[AGIPD][CORRECT] Use calcat_interface and remove precorrection notebook

Description

Updating AGIPD correction notebooks for using calcat_interface and avoiding calibrationDBRemote and cal_db_interactive

Note: This MR is based on https://git.xfel.eu/detectors/pycalibration/-/merge_requests/773

Related to https://git.xfel.eu/calibration/planning/-/issues/150 and https://git.xfel.eu/calibration/planning/-/issues/168

1. Remove notebooks/AGIPD/AGIPD_Retrieve_Constants_Precorrection.ipynb
2. Remove cal_tools/agipdlib functions for loading constants from YAML or DB. Also, remove related tests.
3. Move to retrieve and initialize constants to the AGIPD correction notebook.
4. Use a new feature for displaying the markdown table for retrieved constants.
5. Fix a bug when displaying one retrieved constant in CalibrationData metadata object.
6. Use a fragment file to store retrieved constants.
7. Clean-ups around related code.

How Has This Been Tested?

• Testing..

Relevant Documents (optional)

Types of changes

• New feature (non-breaking change which adds functionality)
• Bug fix (non-breaking change which fixes an issue)

Checklist:

Reviewers

@schmidtp @kluyvert

src/cal_tools/calcat_interface.py

@@ -375,7 +375,6 @@ class CalibrationData:
         modules=None,
         client=None,
         event_at=None,
-        pdu_snapshot_at=None,
         module_naming="da",
     ):
         """Initialize a new CalibrationData object.
@@ -392,8 +391,6 @@ class CalibrationData:
                 communication, global one by default.
             event_at (datetime, date, str or None): Default time at which the
                 CCVs should have been valid, now if omitted
-            pdu_snapshot_at (datetime, date, str or None): Default time of
-                database state to look at, now if omitted.
             module_naming (str or None): Expected module name convention to be
                 used as metadata dict keys. Expected values are:
                 `da`: data aggregator name is used. Default.
@@ -408,7 +405,7 @@ class CalibrationData:
         self.detector_name = detector_name
         self.modules = modules
         self.event_at = event_at
-        self.pdu_snapshot_at = pdu_snapshot_at
+        self.pdu_snapshot_at = event_at
         self.module_naming = module_naming
 
         if client is None:
@@ -932,7 +929,6 @@ class AGIPD_CalibrationData(SplitConditionCalibrationData):
         modules=None,
         client=None,
         event_at=None,
-        pdu_snapshot_at=None,
         gain_setting=None,
         gain_mode=None,
         module_naming="da",
@@ -946,7 +942,6 @@ class AGIPD_CalibrationData(SplitConditionCalibrationData):
             modules,
             client,
             event_at,
-            pdu_snapshot_at,
             module_naming,
         )
 
@@ -1010,7 +1005,6 @@ class LPD_CalibrationData(SplitConditionCalibrationData):
         modules=None,
         client=None,
         event_at=None,
-        pdu_snapshot_at=None,
         module_naming="da",
     ):
         super().__init__(
@@ -1018,7 +1012,6 @@ class LPD_CalibrationData(SplitConditionCalibrationData):
             modules,
             client,
             event_at,
-            pdu_snapshot_at,
             module_naming,
         )
 
@@ -1119,7 +1112,6 @@ class JUNGFRAU_CalibrationData(CalibrationData):
         modules=None,
         client=None,
         event_at=None,
-        pdu_snapshot_at=None,
         module_naming="da",
     ):
         super().__init__(
@@ -1127,7 +1119,6 @@ class JUNGFRAU_CalibrationData(CalibrationData):
             modules,
             client,
             event_at,
-            pdu_snapshot_at,
             module_naming,
         )
 
@@ -1180,7 +1171,6 @@ class PNCCD_CalibrationData(CalibrationData):
         modules=None,
         client=None,
         event_at=None,
-        pdu_snapshot_at=None,
         module_naming="da",
     ):
         # Ignore modules for this detector.
@@ -1189,7 +1179,6 @@ class PNCCD_CalibrationData(CalibrationData):
             modules,
             client,
             event_at,
-            pdu_snapshot_at,
             module_naming,
         )
 
@@ -1236,7 +1225,6 @@ class EPIX100_CalibrationData(SplitConditionCalibrationData):
         modules=None,
         client=None,
         event_at=None,
-        pdu_snapshot_at=None,
         module_naming="da",
     ):
         # Ignore modules for this detector.
@@ -1245,7 +1233,6 @@ class EPIX100_CalibrationData(SplitConditionCalibrationData):
             modules,
             client,
             event_at,
-            pdu_snapshot_at,
             module_naming,
         )
 
@@ -1286,7 +1273,6 @@ class GOTTHARD2_CalibrationData(CalibrationData):
         modules=None,
         client=None,
         event_at=None,
-        pdu_snapshot_at=None,
         module_naming="da",
     ):
         # Ignore modules for this detector.
@@ -1295,7 +1281,6 @@ class GOTTHARD2_CalibrationData(CalibrationData):
             modules,
             client,
             event_at,
-            pdu_snapshot_at,
             module_naming,
         )