Adding Rich variables for Functors
Add pending Rich variables to become Functors, following the list from (PID CALIBRATION STATUS, 18th of May):
- Rich1GasUsed
- Rich2GasUsed
- RichThresholdEl
- RichThresholdKa
- RichThresholdMu
- RichThresholdPi
- RichThresholdPr
- RichThresholdDe
First in ProtoParticle.h, then moving to ProtoParticle.cpp and finally ChargedProtoParticleAddRichInfo.cpp to propagate it.
See also Rec!3563 (closed)
suggested functors added to and superseded by Rec!3624 (merged)
Merge request reports
Activity
requested review from @mveghel
added RTA label
To avoid duplication within
ProtoParticle
, could you directly access theRichPID
object within the functor (Rec!3563 (closed))? This is a great opportunity to do that, as we can then probably move more of this kind of info to the origin (doens't have to be in this MR)Edited by Maarten Van Veghelfyi @jonrob
One additional comment is you have missed one of the mass hypothesis the RICh considers, Deuteron.
https://gitlab.cern.ch/lhcb/LHCb/-/blob/master/Event/RecEvent/include/Event/RichPID.h#L212
Please add this along side the others.
mentioned in merge request Rec!3563 (closed)
added Event model label
added RICH label
mentioned in issue Moore#659 (closed)
mentioned in merge request Rec!3624 (merged)