[MAPPING][DARK][INJECT] PT2 Update injecting functions and Dark notebooks
Summary:
Previously we were using the information about the physical detector units (PDUs) to inject the new generated calibration constant versions (CCVs). With the new detector mapping, PDU will still be used from a different source than the stored device
metadata from cal_db_interactive
.
db_module is the used term for Physical Detector Unit in pyCalibration. db_module will be retrieved if it was not given through myMDC or calibration_configurations in the nb's 1st Cell.
Then it will be used in sending constants to the database and storing them locally.
- I have updated the main pyCalibration wrapper functions @
tools.py
that are used to inject CCVs - I have also updated the main dark notebooks for all 8 detectors (AGIPD, LPD, DSSC, Epix10K, Epix100, pnCCD, JungFrau, and FastCCD) and AGIPD's PC 3- Added report_path as a new sent metadata.
This merge request is connected to:
- https://git.xfel.eu/gitlab/detectors/cal_db_interactive/merge_requests/58
- https://git.xfel.eu/gitlab/detectors/cal_db_interactive/merge_requests/59
- https://git.xfel.eu/gitlab/detectors/cal_db_interactive/merge_requests/60
- https://git.xfel.eu/gitlab/detectors/pycalibration/merge_requests/413
Tests:
- More tests are in progress
Reviewers:
Note: This MR will be rebased on multiple MR's waiting for review.
Merge request reports
Activity
added Waiting for review label
- Resolved by Karim Ahmed
- Resolved by Karim Ahmed
- Resolved by Karim Ahmed
- Resolved by Karim Ahmed
- Resolved by Karim Ahmed
- Resolved by Karim Ahmed
- Resolved by Karim Ahmed
- Resolved by Karim Ahmed
added 14 commits
-
439b8e0e...e0a90b84 - 13 commits from branch
feat/v2_mapping_pt1_retrive_and_corrections
- 46cc167e - resolve conflicts
-
439b8e0e...e0a90b84 - 13 commits from branch
- Resolved by Karim Ahmed
- Resolved by Karim Ahmed
- Resolved by Karim Ahmed
- Resolved by Karim Ahmed
- Resolved by Karim Ahmed
- Resolved by Karim Ahmed
- Resolved by Karim Ahmed
added 1 commit
- 1aa89037 - fix pnccd conflicts, agipd error and remove unneeded Optional
added 1 commit
- e8417deb - update get_pdu_from_detectors based on MR suggestion
added 1 commit
- e8417deb - update get_pdu_from_detectors based on MR suggestion
added 1 commit
- 67afb068 - fixes, updating pnccd gain and updating the requirements
- Resolved by Karim Ahmed
- Resolved by Karim Ahmed
added 9 commits
-
05157353...c332e608 - 8 commits from branch
master
- 1b5a4664 - resolve conflicts
-
05157353...c332e608 - 8 commits from branch
Offline LGTM from @danilevc
mentioned in commit d5898d20
removed Waiting for review label