Update RICH decoding for realistic cable maps from the pit
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.
Merge request reports
Activity
mentioned in merge request Boole!340 (merged)
added 1 commit
- a194b9c3 - Update RICH script to create mapping conditions
added 1 commit
- 48bac33c - RichDecoder - Do not make bit->anode lookup failure a hard assert error
added 1 commit
- 26f393b0 - RichDecoder - Do not make bit->anode lookup failure a hard assert error
added 1 commit
- f91eaf5d - RichDecoder - Do not make bit->anode lookup failure a hard assert error
added 1 commit
- cedcdf4d - RichDecoder - Do not make bit->anode lookup failure a hard assert error
added 8 commits
-
5b3b8387...bdc20132 - 2 commits from branch
lhcb:master
- cdc8a4f9 - Adapt DAQ classes to updated cable mapping from the pit
- a76bd6fc - Update RICH script to create mapping conditions
- ab95c91c - RichDAQDefinitions - Minor changes to rawDump print format
- d2763934 - RichDecoder - Do not make bit->anode lookup failure a hard assert error
- 46ca9b9c - RichCommonBase - Add some additional messaging helpers
- 7f3ad592 - RichRawBankDecoder - Update messaging to use new helpers
Toggle commit list-
5b3b8387...bdc20132 - 2 commits from branch
- Resolved by Christopher Rob Jones
HI @jonrob, is this ready to be tested?
mentioned in issue Moore#278 (closed)
added 10 commits
-
7f3ad592...bbcc16b1 - 4 commits from branch
lhcb:master
- 0995dac3 - Adapt DAQ classes to updated cable mapping from the pit
- dd7de65a - Update RICH script to create mapping conditions
- f77c60fd - RichDAQDefinitions - Minor changes to rawDump print format
- b0398475 - RichDecoder - Do not make bit->anode lookup failure a hard assert error
- 0e7a2614 - RichCommonBase - Add some additional messaging helpers
- 098354d4 - RichRawBankDecoder - Update messaging to use new helpers
Toggle commit list-
7f3ad592...bbcc16b1 - 4 commits from branch
assigned to @jonrob
added ci-test-triggered label
added 1 commit
- d202a2b1 - RichCommonBase - Add missing std::forward for variadic template arguments
- [2021-05-19 18:13] Validation started with lhcb-master-mr#2344
- [2021-05-27 13:56] Validation started with lhcb-master-mr#2383
- [2021-05-28 00:26] Validation started with lhcb-head#2955
- [2021-05-28 01:46] Validation started with lhcb-head#2955
- [2021-05-29 00:30] Validation started with lhcb-head#2956
Edited by Software for LHCbadded 6 commits
- 501d270a - Move RICH messaging utilites to standalone header
- e94fa000 - RichFutureKernel - Use RichFutureUtils
- c65b20fc - RichFutureDAQ - A RichFutureUtils include area to CMake dependencies
- 9073d9f6 - Add missing utility include to RichMessaging.h
- 53347cdb - RichRawBankDecoder - Use private instances of decoding derived condition helpers
- 50a8fe7c - Add debug/verbose messaging to DAQ utilities
Toggle commit list