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

Review changes
Download
Patches
Plain diff

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

Overview 28
Commits 23
Pipelines 0
Changes 8

Merged Karim Ahmed requested to merge AGIPD_calcat_interface into master 2 years ago

Overview 28
Commits 23
Pipelines 0
Changes 8

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

Remove notebooks/AGIPD/AGIPD_Retrieve_Constants_Precorrection.ipynb
Remove cal_tools/agipdlib functions for loading constants from YAML or DB. Also, remove related tests.
Move to retrieve and initialize constants to the AGIPD correction notebook.
Use a new feature for displaying the markdown table for retrieved constants.
Fix a bug when displaying one retrieved constant in CalibrationData metadata object.
Use a fragment file to store retrieved constants.
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

Edited 1 year ago by Karim Ahmed

Merge request reports

Activity

Filter activity

Approvals
Assignees & reviewers
Comments (from bots)
Comments (from users)
Commits & branches
Edits
Labels
Lock status
Mentions
Merge request status
Tracking

Please register or sign in to reply

0 Assignees

None

Select assignees

0 Reviewers

Request review from

Labels

None

Select labels

Manage project labels

Milestone

None

Time tracking

No estimate or time spent

0 Participants

There are currently no pipelines.

To run a merge request pipeline, the jobs in the CI/CD configuration file must be configured to run in merge request pipelines and you must have sufficient permissions in the source project.