[PWGJE] Adding reflection signal from MC and updating jet matching process - #17750
[PWGJE] Adding reflection signal from MC and updating jet matching process#17750Rajdeep-nandi wants to merge 13 commits into
Conversation
… issues, remove unused includes, apply clang-format
Please consider the following formatting changes to AliceO2Group#17750
|
@Rajdeep-nandi The PR title is too vague. Please add info about the location. |
|
O2 linter results: ❌ 0 errors, |
|
Error while checking build/O2Physics/code-check for 80065c8 at 2026-09-04 12:32: Full log here. |
80065c8 to
cd35e78
Compare
Please consider the following formatting changes to AliceO2Group#17750
|
Error while checking build/O2Physics/code-check for 206670e at 2026-09-07 10:39: Full log here. |
|
|
||
| const auto d0mcdJetsPerCollision = mcdjets.sliceBy(d0MCDJetsPerCollisionPreslice, collision.globalIndex()); | ||
| for (const auto& mcdjet : d0mcdJetsPerCollision) { | ||
| if (mcdjet.pt() <= jetPtMin || mcdjet.r() != jetRBin) { |
There was a problem hiding this comment.
does this jet radius comparison work? I am not 100% sure, but i thought jet R is float and you compare to an int
There was a problem hiding this comment.
Dear @fjonasALICE
DECLARE_SOA_COLUMN(R, r, int); //!PWGJE/DataModel/Jet.h
So .r() returns an int, it stores as R * 100 rounded to an integer (e.g. R = 0.4 → r() == 40)
Thatswhy i used it as int
Thank you
| } | ||
| registry.fill(HIST("MC_CollisionCounter"), getValFromBin(BinMCColCntr::SelectedMCCollisions)); | ||
|
|
||
| // const float mcWeight = mccollision.weight(); |
There was a problem hiding this comment.
is it intentional that you no longer fill any histogram with any weights?
There was a problem hiding this comment.
Yes, It was intentional because I think the HF MC are usually not weighted
This pull request includes :