[PWGCF] nchCumulantsId: Make axes configurable#16368
Conversation
Removed commented-out code for various track quality assessment functions and their calls, streamlining the nchCumulantsId.cxx file.
Removed unused variables related to electron and deuteron track identification.
additional Event cuts has been added
additional checks has been added
Additional checks has been added
Additional checks has been added
Removed unused variables related to electron and deuteron track identification.
Added configurable rejection cuts for PID selection and updated histogram registration for PID validation.
Bins corrected for Corrections
pravatp98
requested review from
SwatiSaha-1997,
alibuild,
isputows,
otonvd,
shouqiye and
victor-gonzalez
as code owners
|
O2 linter results: ❌ 0 errors, |
github-actions
Bot
changed the title
|
Dear @SwatiSaha-1997 @victor-gonzalez |
|
@pravatp98, even if the PR is approved by reviewers, it will not be merged as |
|
Dear @SwatiSaha-1997 , and the macOs-arm error doesn't seem to be coming from my task. Anton Alkin replied to the error query that the error was the issue of the O2 framework. Here is the link. Kindly please approve my PR. Best Regards, |
|
Dear @SwatiSaha-1997 |
vkucera
left a comment
vkucera
left a comment
There was a problem hiding this comment.
Dear @SwatiSaha-1997 , the headers are now corrected
No, they are not. The current version of the PR does not touch the includes at all.
vkucera
changed the title
|
@pravatp98 Please use meaningful PR titles. "Net charge fluctuations3" does not tell me anything about what you did. |
Header files are corrected. Configurable axis are inserted for Net species fluctuation and correlation analysis
…Headers added Required headers added also configurable axis added to the analysis
Please consider the following formatting changes to AliceO2Group#16368
Configurable axis added with fixed header issues
4 participants
Configurable bins added