-
Notifications
You must be signed in to change notification settings - Fork 613
[ALICE3] Add configuration indices to OnTheFly #13856
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
|
O2 linter results: ❌ 80 errors, |
|
@njacazio from my side I think that things should be ready to merge if you have no objections. In principle, more histograms from the tracker task should be moved to the dynamic histograms folders that are created but this can be changed in future pr's I believe. I made the change for the most important ones anyways |
|
Thanks @jesgum this is ok for the single configuration. The selection should be done everywhere. |


Change the configurable for the luts to be a vector where each element results in a new configuration that will be processed by the otf workflow. This means that each collision will have multiple instances of the same track (one for each config), and these can be separated by an index getter from a new table.
In general I'm not sure how good of an idea it is to have tracks from all different configurations to be associated with the same collision. @njacazio @ddobrigk please have a look and tell me what you think.