Repository navigation
[PWGCF] Net charge fluctuations Update3: Updated the Particle Identification in Code - #18676
Conversation
Updated for efficiency correction and sub-sampling
…ficiency correction Refactor identification bits to use constexpr instead of #define, update includes to comment out unused headers, and improve readability of various functions.
Commented out unused identification bits for clarity.
Added configurable pT cutoffs for kaons and protons, and updated relevant sections to use these cutoffs for efficiency corrections.
|
O2 linter results: ❌ 0 errors, |
|
Error while checking build/O2Physics/code-check for b440ab7 at 2026-11-10 07:02: Full log here. |
|
Error while checking build/O2Physics/code-check for d8d29eb at 2026-11-10 07:31: Full log here. |
|
Error while checking build/O2Physics/code-check for 32e84b7 at 2026-11-10 18:59: Full log here. |
|
Error while checking build/O2Physics/o2 for 32e84b7 at 2026-11-10 19:08: Full log here. |
|
Error while checking build/O2Physics/staging for 32e84b7 at 2026-11-10 20:27: Full log here. |
|
@pravatp98 You are supposed to test your changes locally before opening a PR, which you apparently did not do, since you are including a header that does not exist. |
|
@pravatp98 Please fix the PR title. It does not say anything about what you did. See the contribution guidelines. https://aliceo2group.github.io/analysis-framework/docs/gettingstarted/contributingtocode.html#pull-requests |
thank you vit. thank you for poinitng this out, i will keep this in mind, Since the earlier issue was one header file missing i just kept it. i have fixed that as per your suggestion |
Dear @victor-gonzalez @SwatiSaha-1997 @dsarkaralice @isputows @fsii
I have updated the code for the Analysis. As you can see it has passed all checks so kindly review and approve the PR
With best Regards
Pravat
|
|
Dear @pravatp98, you should request Iwona and Fan for approval of EbyE PRs. They are in charge of this. |

Updated for Particle Identification