-
Notifications
You must be signed in to change notification settings - Fork 613
[Common] Centrality task: add PV mult. binning to histogram configurations #13543
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: ❌ 2 errors, |
|
Hi @ercolessi, thanks, of course no objection - though note that the compiler is shouting due to some missing semicolons, I believe. Can you please take a look? I can approve once fixed. Then for the rest indeed at some point we should check how to best move forward :-) |
|
Error while checking build/O2Physics/o2 for c4eea82 at 2025-10-25 16:28: Full log here. |
|
Error while checking build/O2Physics/o2 for 34f9154 at 2025-10-25 18:13: Full log here. |
|
Hi @ddobrigk indeed my apologies it was a silly mistake, thanks for the approval! |
…tions (AliceO2Group#13543) Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
…tions (AliceO2Group#13543) Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
Hi @ddobrigk!
I’ve been asked to produce a calibration for the LF MC centrality for OO.
To do this, I’ll need some updates to the old centrality task (these histograms were not ported to the new multCent task).
Given the urgency (Alice Physics Week), I’d like to request these changes now if you agree. However, as we discussed previously, I am happy to chat on how to handle this more efficiently going forward. 🙂
Thanks a lot!