-
Notifications
You must be signed in to change notification settings - Fork 613
[PWGLF] added QC task for nuclei, moved nucleiUtils.h to the appropriate folder #13212
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
GiorgioAlbertoLucia
commented
Oct 2, 2025
- nucleiQC.cxx is added for QC of nuclei and producing output tables upon request for studies of primary fractions (this will be used for the p-He3 analysis)
- A small table is added for this reason to LFSlimNucleiTable.h
- nucleiUtils.h is moved too the Utils folder, paths in tasks depending on this header file is updated accordingly
- new utilities are added to nucleiUtils.h
…ion selection and histogram registry generator
|
O2 linter results: ❌ 95 errors, |
Please consider the following formatting changes to AliceO2Group#13212
|
Error while checking build/O2Physics/o2 for 1f13e26 at 2025-10-03 09:48: Full log here. |
Please consider the following formatting changes to AliceO2Group#13212
Head branch was pushed to by a user without write access
|
@GiorgioAlbertoLucia @maciacco Please do not leave PRs which do not pass required checks as "ready for review". |
|
Hi @vkucera, thank you for your comment. I think the change that made the PR formatting fail happened during the last alibuild merge and we were not expecting this behavior. We will be more cautious next time! |
|
@GiorgioAlbertoLucia @maciacco The new |
…ate folder (AliceO2Group#13212) Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
…ate folder (AliceO2Group#13212) Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
…ate folder (AliceO2Group#13212) Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
…ate folder (AliceO2Group#13212) Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
| using namespace o2; | ||
| using namespace o2::framework; | ||
| using namespace o2::framework::expressions; | ||
| using namespace o2::constants::physics; |
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.
How about fixing these?
…ate folder (AliceO2Group#13212) Co-authored-by: ALICE Action Bot <alibuild@cern.ch>