[AGIPD][DARK] Improvements for reading conditions by creating new dataclass for multiple runs

Description

While working on a feature to sort dark runs for AGIPD, I realized that the AGIDP dark notebook might need some improvements. This is a refactor to AGIPD darks with no new functionalities.

1- Introduce AgipdCtrlRuns data class

2- Convert AgipdCtrl class to dataclass

3- Replace cells for reading/validating AGIPD Conditions with AgipdCtrlRuns

4- Now I add more tests to untested pieces in the dark notebook.

This leads to simpler code and better performance while reading conditions. This can also be used with other notebooks, e.g. Gain notebooks that reads more than one run.

How Has This Been Tested?

• TBD

Relevant Documents (optional)

Types of changes

• Refactor (refactoring code with no functionality changes)
• Test (additional or refactored tests)
• Chore (non-code changes, e.g. comments, readme, grammar, etc...)

Checklist:

• Add unit tests

Reviewers

@kluyvert @schmidtp

added Doing label

assigned to @ahmedk

added 1 commit

• 1666dbdd - Introduce new AgipdCtrlDark Class and use dataclass for AgipdCtrl

Compare with previous version

changed title from Draft: [AGIPD][DARK] Introduce new AgipdCtrlDark Class and use dataclass for AgipdCtrl to Draft: [AGIPD][DARK] Introduce new AgipdCtrlRuns Class and use dataclass for AgipdCtrl

added 1 commit

• 7a545c91 - Rename AGIPDCtrlDark with AgipdCtrlRuns

Compare with previous version

changed the description

added 2 commits

• 02e75acb - add tests
• cb43072c - Separate gain mode validation into validate_gain_modes

Compare with previous version

added 1 commit

• ed3004c3 - apply isort on test_agipdlib

Compare with previous version

added 2 commits

• 79cf9be1 - Change AgipdCtrlsRuns arguments to raw_folder and runs for usability
• 2bb639f5 - fix assert positions

Compare with previous version

marked this merge request as ready

changed title from Draft: [AGIPD][DARK] Introduce new AgipdCtrlRuns Class and use dataclass for AgipdCtrl to [AGIPD][DARK] Improvements for reading conditions by creating new dataclass for multiple runs

marked the checklist item Add unit tests as completed

added Testing Waiting for review labels and removed Doing label

changed the description

marked the checklist item TBD as completed

added 1 commit

• d785b3cc - isort agipdlib

Compare with previous version

added 1 commit

• 225def4c - fix get number of runs

Compare with previous version