Skip to content

ITS: allow to reconstruct incomplete track topologies#15390

Merged
f3sch merged 2 commits into
AliceO2Group:devfrom
f3sch:its/holes
May 15, 2026
Merged

ITS: allow to reconstruct incomplete track topologies#15390
f3sch merged 2 commits into
AliceO2Group:devfrom
f3sch:its/holes

Conversation

@f3sch
Copy link
Copy Markdown
Collaborator

@f3sch f3sch commented May 11, 2026

Generalizes across which layers combinatorics are built via new class TrackingTopology and LayerMask. This allows to built track topology with holes. Tested some pp and Pb-Pb data, which gives same output as before (by default this is off). There are two new confKeys maxHolesIter and holeLayerMaskIter steering on which layers a holes is allowed and how many in total.

Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
@f3sch f3sch marked this pull request as ready for review May 12, 2026 04:57
Comment thread Detectors/ITSMFT/ITS/tracking/src/TimeFrame.cxx
@f3sch
Copy link
Copy Markdown
Collaborator Author

f3sch commented May 13, 2026

@mpuccio, when you can please have a look if this is fine for you

Copy link
Copy Markdown
Contributor

@mpuccio mpuccio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @f3sch, while I can't say that I went through with very high precision, this seems an elegant solution, so all good for me! Thanks!

Comment thread Detectors/ITSMFT/ITS/tracking/GPU/cuda/TrackerTraitsGPU.cxx Outdated
@f3sch f3sch requested a review from fprino May 15, 2026 07:22
@f3sch
Copy link
Copy Markdown
Collaborator Author

f3sch commented May 15, 2026

Hi @shahor02 @fprino, Max is fine with the changes so we could merge if we wanted :)

@f3sch f3sch merged commit 877dc1f into AliceO2Group:dev May 15, 2026
13 checks passed
@f3sch f3sch deleted the its/holes branch May 15, 2026 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants