Skip to content

Conversation

@jesgum
Copy link
Contributor

@jesgum jesgum commented Nov 17, 2025

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.

  • All the luts provided will be loaded into memory
  • If for some reason more luts are provided for one particle species than for others, when we "run out" of tables, we'll default to the first one provided.
  • If only one configuration is used, everything should be remained unchanged and subsequent analysis tasks should run fine without requiring any change.

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.

@github-actions github-actions bot changed the title Add configuration indices to OnTheFly [ALICE3] Add configuration indices to OnTheFly Nov 17, 2025
@github-actions
Copy link

github-actions bot commented Nov 17, 2025

O2 linter results: ❌ 80 errors, ⚠️ 90 warnings, 🔕 0 disabled

@jesgum
Copy link
Contributor Author

jesgum commented Nov 18, 2025

I ran a local cross-check to see that if I only run one configuration, the current behaviour is well reproduced. I also checked
the performance with the current tracker run twice and the new one once with two configs.
See figures:

combined_plots_ptres-1 combined_plots_dcaxy-1

@jesgum jesgum marked this pull request as ready for review November 19, 2025 17:20
@jesgum
Copy link
Contributor Author

jesgum commented Nov 19, 2025

@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

@njacazio
Copy link
Collaborator

Thanks @jesgum this is ok for the single configuration. The selection should be done everywhere.

@njacazio njacazio enabled auto-merge (squash) November 19, 2025 17:24
@njacazio njacazio merged commit fc54091 into AliceO2Group:master Nov 21, 2025
12 of 14 checks passed
yakparo pushed a commit to yakparo/O2Physics that referenced this pull request Nov 21, 2025
@jesgum jesgum deleted the multi-conf branch November 24, 2025 13:54
lmattei01 pushed a commit to lmattei01/O2Physics that referenced this pull request Dec 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants