Skip to content

[ALICE3] Fixes for OTF Multicharm #14587

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.

Sign up for GitHub

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

Jump to bottom
Open
jesgum wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[ALICE3] Fixes for OTF Multicharm #14587

jesgum wants to merge 2 commits into from
+214 −185

Conversation

@jesgum
Copy link
Contributor

@jesgum jesgum commented Jan 22, 2026

No description provided.

@github-actions github-actions bot changed the title Fixes for OTF Multicharm [ALICE3] Fixes for OTF Multicharm Jan 22, 2026
@github-actions
Copy link

github-actions bot commented Jan 22, 2026 •
edited
Loading

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

@jesgum jesgum marked this pull request as ready for review January 22, 2026 14:33
@jesgum
Copy link
Contributor Author

jesgum commented Jan 22, 2026 •
edited
Loading

@njacazio I think that this could be merged for now. In the end, did we move the lutConfigId to collision level instead of track level yet? If so I would have to adjust for that as well but will do so in a separate pr. Otherwise I could take a look at it soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alibuild alibuild Awaiting requested review from alibuild alibuild is a code owner

@njacazio njacazio Awaiting requested review from njacazio njacazio is a code owner

@hscheid hscheid Awaiting requested review from hscheid hscheid is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

alice3

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@jesgum