Skip to content

Manual sweep of !27924 to 21.9

Nicolas Koehler requested to merge nkoehler/athena:mmCnv into 21.9

Hi,

this MR is a manual sweep of !27924 (merged) to 21.9 (since the automatic one failed). It changes the way MMPrepDataContainerCnv_p1 write/reads the MMPrepData objects. The Muon::MMPrepDataContainer_p1->m_prdDeltaId vector stored the difference between cluster identifier and collection (module) identifier. However, this difference became larger than the maximum of unsigned short which is the data type of m_prdDeltaId (i.e. for multilayer=2 clusters). This resulted in the Diff is greater than max size of diff permitted!!! warning message printed by MMPrepDataContainerCnv_p1.

Now, we encode multilayer, gasGap and channel in the unsigned short bits in transToPers and read it back in persToTrans. So we can encode all possible cluster identifiers and have also MMPrepData for multilayer=2 when reading ESDs.

Note: All currently existing Run 3 ESDs made with the previous version of MMPrepDataContainerCnv_p1 (only validation samples) need to be redone (we will do this in muon software centrally).

Best, Nico

Merge request reports