-
Notifications
You must be signed in to change notification settings - Fork 488
ITSMFT: scaffolding for staggered clusterization #15004
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
Conversation
Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
|
Error while checking build/O2/fullCI_slc9 for 14781d4 at 2026-01-30 07:34: Full log here. |
|
@shahor02 when you have time, it would be good to have your input :) |
shahor02
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, only insignificant comments
Detectors/ITSMFT/common/reconstruction/include/ITSMFTReconstruction/ClustererParam.h
Outdated
Show resolved
Hide resolved
|
@shahor02, I implemented your comment, so I think we are good to merge :)? |
|
Sure |
Adds scaffolding for staggered clusterization.
Also combines the superfluous separated ClustererSpec/Writer for MFT&ITS into one common templated one.
Since staggering is hidden behind compilation flags for now, I tested that I get the same number of clusters&tracks before and after applying this patch.