Skip to content

MuonChamberHoleRecovery - Ensure that drift circles are unique

Johannes Junggeburth requested to merge jojungge/athena:MuPatDuplHit into main

For one reason or another, the can be 2 hits for the same Mdt tube in the PrepData collection.

MuPatTrackBuilder.Mu...MuonChamberHoleRecoveryTool    22    21 SUCCESS Two hits recorded for the same tube
*** current: MDT BIS eta -3 phi  6 ml 2 lay 3 tube 14  r  4617 z -2443 phi -1.907 theta 2.057 lPos 14.89 error  0.07  r_drift 14.89
*** previous: MDT BIS eta -3 phi  6 ml 2 lay 3 tube 14  r  4624 z -2430 phi -1.907 theta 2.055 lPos  0.22 error  0.23  r_drift  0.22
MuPatTrackBuilder.Mu...MuonChamberHoleRecoveryTool    30    23 SUCCESS Two hits recorded for the same tube
*** current: MDT BOS eta  1 phi  2 ml 1 lay 1 tube 16  r 10386 z   496 phi 1.267 theta 1.523 lPos 14.89 error  0.07  r_drift 14.89
*** previous: MDT BOS eta  1 phi  2 ml 1 lay 1 tube 16  r 10386 z   483 phi 1.267 theta 1.524 lPos  1.69 error  0.16  r_drift  1.69
MuPatTrackBuilder.Mu...MuonChamberHoleRecoveryTool    30    23 SUCCESS Two hits recorded for the same tube
*** current: MDT BOS eta  1 phi  2 ml 1 lay 3 tube 16  r 10438 z   496 phi 1.267 theta 1.523 lPos 14.89 error  0.07  r_drift 14.89
*** previous: MDT BOS eta  1 phi  2 ml 1 lay 3 tube 16  r 10438 z   487 phi 1.267 theta 1.524 lPos  5.72 error  0.08  r_drift  5.72

If none of the hits is picked-up by the segment maker, the chamber hole recovery puts both back on track, triggering a duplicate hit warning from the GlobalChiFitter. This MR introduces a proper treatment by choosing the hit giving the better residual. I let the decision to @kluit, @stavrop to investigate the underlying reason at the bytestream converter level.

Merge request reports