-
Notifications
You must be signed in to change notification settings - Fork 433
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
ITS tracking works on generic N layers #4511
Conversation
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.
I see no MFT or common code involved.
82137a8
to
79d6dc5
Compare
Please consider the following formatting changes to AliceO2Group#4511
Hi @davidrohr, @iouribelikov |
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.
Fine with me, I have seen that the o2 CI is currently generally failing but fullCI and mac have passed, so it should be OK. If @iouribelikov approves as well, we can squash-merge it.
Thanks @iouribelikov !! |
This introduces the changes required to run the ITS tracking suite on dataset with a different number of layers (ITS3 with more than 3 layers in the IB, ALICE3...).
@mconcas please check that everything is fine with you.
@qgp FYI