Skip to content
Snippets Groups Projects

Update RICH decoding for realistic cable maps from the pit

Merged Christopher Rob Jones requested to merge jonrob/LHCb:rich-update-cable-maps into master

Updates the RICH decoding to adapt to slight changes in the mappings introduced in lhcb-conddb/SIMCOND!148 (merged)

Note the realistic encoding and decoding is not used by default anywhere, the only place it is run is a specific QM test in Boole (which will get its own MR to update). This was intentional as I knew the original format would not completely match the format eventually deployed at the pit. So there are no back-wards compatibility issues.

Should be merged with Boole!340 (merged) Lbcom!557 (merged) Panoptes!122 (closed)

Tested now with commissioning data, and looks OK.

Screenshot_2021-05-27_at_10.44.41_am Screenshot_2021-05-27_at_10.45.16_am

Edited by Christopher Rob Jones

Merge request reports

Pipeline #2653922 passed

Pipeline passed for feddc55a on jonrob:rich-update-cable-maps

Approval is optional

Merged by Christopher Rob JonesChristopher Rob Jones 3 years ago (May 29, 2021 9:23am UTC)

Merge details

  • Changes merged into with 570c97b2.
  • Deleted the source branch.

Pipeline #2662291 passed

Pipeline passed for 570c97b2 on master

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