Skip to content

Add fixed r22 track selections

#96 (closed) describes the problems with the current track selections. We discussed there moving the track selection out of this package and pushing it downstream, instead saving several bool flags for different track selections. Since that will require a change to all downstream packages (and break backward compatibility anyway) this MR takes the more cautious approach of just adding a new r22default track selection. If we don't expect the track selection to change many times it may not make sense to do this business with the flags.

See also atlas/athena!58144 (merged) which describes the naming:

  • R22_DEFAULT, which is the same as the "dips loose" track selection but increases the min Si hit from 7 -> 8 (in line with recommendations)
  • R22_LOOSE, which is the same as R22_DEFAULT but removes the shared module cut and relaxes the d0 cut. This has also been called "gnn loose" selection in the group (but I think it probably makes sense to avoid naming selections on the algorithms which were first used to study them)

Tagging @dguest @vdao @mguth @pgadow @alfroch

Merge request reports