-
Notifications
You must be signed in to change notification settings - Fork 498
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
Added three-particle correlations task #6988
Conversation
Please consider the following formatting changes to AliceO2Group#6988
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.
Thanks for moving forward this functionality!
As O2Analysis
and Task
are implicit from the position in the source tree, please rename the source file as just ThreeParticleCorrelations.cxx
PWGCF/MultiparticleCorrelations/Tasks/O2AnalysisTaskThreePartCorr.cxx
Outdated
Show resolved
Hide resolved
PWGCF/MultiparticleCorrelations/Tasks/O2AnalysisTaskThreePartCorr.cxx
Outdated
Show resolved
Hide resolved
PWGCF/MultiparticleCorrelations/Tasks/O2AnalysisTaskThreePartCorr.cxx
Outdated
Show resolved
Hide resolved
PWGCF/MultiparticleCorrelations/Tasks/O2AnalysisTaskThreePartCorr.cxx
Outdated
Show resolved
Hide resolved
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.
Thanks!
Please, have a look at my suggestions for the next iteration in case you wanted to consider them
Error while checking build/O2Physics/o2 for 1a113ee at 2024-07-31 15:26:
Full log here. |
Please consider the following formatting changes to AliceO2Group#6988
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.
The compiler errors haven't been fixed (not all of them, at least), it will fail again
Please consider the following formatting changes to AliceO2Group#6988
* Added revised three-particle correlation function * Added a null output to V0Sign() * Added a LOGF-statement in processMixed() * Please consider the following formatting changes --------- Co-authored-by: ALICE Action Bot <[email protected]>
* Added revised three-particle correlation function * Added a null output to V0Sign() * Added a LOGF-statement in processMixed() * Please consider the following formatting changes --------- Co-authored-by: ALICE Action Bot <[email protected]>
* Added revised three-particle correlation function * Added a null output to V0Sign() * Added a LOGF-statement in processMixed() * Please consider the following formatting changes --------- Co-authored-by: ALICE Action Bot <[email protected]>
No description provided.