Preliminary changes to support using RICH PMT/Module/Panel classes in the reconstruction

Should be tested and merged with LHCb!3446 (merged) Rec!2759 (merged)

Will also requires changes to the decoding mapping in lhcb-conddb/lhcb-conditions-database!13 (merged)

@seaso @bcouturi @bmalecki @tlin FYI

added RICH label

mentioned in merge request LHCb!3446 (merged)

changed the description

added 1 commit

• 93d283cc - Add RICH PMT NumPixels and PixelArea accessors

Compare with previous version

added 2 commits

• c90c04f2 - Add centralised dd4hep printout of RICH objects
• f25f186f - remove (more) couts from UT

Compare with previous version

added 1 commit

• b106ea69 - remove (more) couts from UT

Compare with previous version

added 3 commits

• 5692ded2 - Add centralised dd4hep printout of RICH objects
• cefb13c8 - Suppress more couts from UT
• 2a6f039b - Add 'run' file to git ignore list

Compare with previous version

added 1 commit

• 98d4f050 - Add more flexible, safer API for accessing RICH dd4hep parameters

Compare with previous version

added 17 commits

• 98d4f050...20359137 - 11 commits from branch lhcb:master
• 9a91fa9c - Preliminary changes to support using PMT/Module/Panel classes in the reconstruction
• 9515b19e - Add RICH PMT NumPixels and PixelArea accessors
• 4a91c8fd - Add centralised dd4hep printout of RICH objects
• 1715102d - Suppress more couts from UT
• 1868e728 - Add 'run' file to git ignore list
• 22ffd08c - Add more flexible, safer API for accessing RICH dd4hep parameters

Compare with previous version

added 3 commits

• ee010beb - DeRichMapmt: Use smaller ints for data members where appropriate
• cf4b48bd - RichPmtGeoAux: clean up
• 1485b0d0 - use auto to access static entities

Compare with previous version

added 2 commits

• 326a57e2 - Rich2MirrorGeoAux: clean up
• 92738ac9 - Reduce size of various DeRichX int data members where appropriate

Compare with previous version

added 1 commit

• 774e96c8 - Remove obsolete RICH PMTDetectorNumbers conditions

Compare with previous version

added 1 commit

• 179d3b88 - suppress some clang warnings

Compare with previous version

added 1 commit

• a699e680 - revert back to 32bit ints for now

Compare with previous version

added 3 commits

• 09e49413 - DeIOV.h: Add more support for unit translations
• 2ad4b6dc - Adapt to change in name of unit change method
• 7710460e - Rich: Convert to 'LHCb' units where necessary on the fly

Compare with previous version

This is excellent; BTW, it may be useful to convert Gas pressure also. At the moment, the nominal 1.013 bar is 6.32 X 10^8 MeV/mm^3 in DD4HEP

Sure, I'll look into pressure units...

changed title from Draft: Preliminary changes to support using PMT/Module/Panel classes in the reconstruction to Draft: Preliminary changes to support using RICH PMT/Module/Panel classes in the reconstruction

added 1 commit

• fd42df5e - RichNotImplemented: Add optional message string

Compare with previous version