I see that the HPTDC ADC/TDC counts are also divided by 4 here: MdtRdoToPrepDataToolMT.cxx However, calibstream MDT bytestream is converted directly to PRD, bypassing RDO, so I would not expect this code to would not be executed for calibstream
Sorry for the late comments, but I was on vacation when this MR started. Anyway, a few comments:
Well, overall I see that the new JSON formats for RT and T0s are just intermediate data structures used only in MdtCalibDbAlg so my suggestions are very critical.
Edward Diehl (5d8f9918) at 16 Mar 22:03
Edward Diehl (5d8f9918) at 16 Mar 21:54
Update README.md accounting for new release numbering
... and 69197 more commits
I'd prefer to leave the existing muonfixedid alone and make a separate NSW muonfixedid. At the muon calibration centers we use muon calibration stream data which does not have NSW data and its very unlikely ever to have NSW data. So I do not anticipate ever have a need for an NSW muonfixedid and would rather not change the existing muonfixedid.