[AGIPD][Dark] Add timings

Description

In this redmine ticket it was realized that AGIPD dark processing lacks timings for different cells. This MR addresses this issue by adding timings for the AGIPD dark notebook as well as the overall summary notebook.

How Has This Been Tested?

• TBD using unit tests.

Relevant Documents (optional)

agipd_timings.pdf

Types of changes

• Bug fix (non-breaking change which fixes an issue)

Checklist:

Reviewers

@tmichela @kluyvert

added Testing label

assigned to @ahmedk

changed title from Fix/add timings agipd to [AGIPD][Dark] Add timings

added Waiting for review label and removed Testing label

marked the checklist item TBD using unit tests. as completed

changed milestone to %3.11.5

changed the description

It shouldn't be necessasry to call .start() again right after .done_step() - each new call to .done_step() is timed since the last step.

There's also no harm in calling .start() again, though.

Thanks! I had been thinking about a more sweeping approach of timing every cell using Princess, but on reflection I prefer this approach - it's more work up front, but it will be easier to make sense of the resulting information when we want to know what's slow.

added 1 commit

• 7b3a148b - improve timing message based on MR suggestion and record timing for async constant retrieval

Compare with previous version

LGTM

Thank you for the review!

merged

mentioned in commit 91b39e32

changed milestone to %3.11.4

removed Waiting for review label