Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adding number of holes to track class #2

Merged
merged 5 commits into from
Feb 27, 2023
Merged

adding number of holes to track class #2

merged 5 commits into from
Feb 27, 2023

Conversation

madbaron
Copy link

@madbaron madbaron commented May 3, 2022

Inspired by today's discussion at the Joint Tracker and Calorimeter Meeting https://agenda.infn.it/event/31242/,
I have added the number of holes to the LCIO track EDM

@madbaron
Copy link
Author

madbaron commented May 5, 2022

Tested at runtime - removing Draft label

@madbaron madbaron marked this pull request as ready for review May 5, 2022 15:44
@kkrizka
Copy link

kkrizka commented May 10, 2022

Would it be possible to implement this per subdetector, similar to TrackImpl::subdetectorHitNumbers? ie: TrackImpl::subdetectorHoleNumbers

@madbaron
Copy link
Author

Hi @kkrizka,

I like the idea.
However, let me turn it into a boomerang question: do you know if it's possible to get the number of holes per subdetector from ACTS?

Right now I am simply retrieving the total number of holes from Acts::MultiTrajectoryHelpers::TrajectoryState:nHoles.

@gianelle gianelle merged commit f611df1 into master Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants