Skip to content
Snippets Groups Projects

Look up CCVs using parameter_name in place of parameter_id

Merged Thomas Kluyver requested to merge feat/calcat-parameter-name into master

Description

In order to find calibration constants, we translate parameter names like 'Sensor Bias Voltage' to numeric IDs in the database. This involves several extra HTTP requests in each correction job. With the next version of CalCat, we can use the names directly in the CCV query instead; CalCat will translate these to IDs internally.

This depends on a change which is merged in CalCat but not yet deployed: https://git.xfel.eu/ITDM/calibration_catalog/-/merge_requests/135

How Has This Been Tested?

TBD

Types of changes

  • Efficiency improvement

Checklist:

  • My code follows the code style of this project.

Reviewers

@manettim @schmidtp

Merge request reports

Loading
Loading

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
Loading