[PWGCF] added a process function for mixed event combinatorial bkg for phi#15710
[PWGCF] added a process function for mixed event combinatorial bkg for phi#15710prottayCMT wants to merge 3 commits intoAliceO2Group:masterfrom
Conversation
github-actions
bot
changed the title
|
O2 linter results: ❌ 64 errors, |
PWGCF/TableProducer/filter2Prong.cxx
Outdated
| // #define FLOAT_PRECISION 0xFFFFFFF0 | ||
| #define O2_DEFINE_CONFIGURABLE(NAME, TYPE, DEFAULT, HELP) Configurable<TYPE> NAME{#NAME, DEFAULT, HELP}; | ||
|
|
||
| namespace o2::aod |
There was a problem hiding this comment.
Move this to the data model, then we do not need to have it twice
PWGCF/TableProducer/filter2Prong.cxx
Outdated
|
|
||
| for (auto it = pairs.begin(); it != pairs.end(); it++) { | ||
| auto& [collision1, tracks1, collision2, tracks2] = *it; | ||
| // float multiplicity = getMultiplicity(collision1); |
There was a problem hiding this comment.
Remove the commented lines
| // float multiplicity = getMultiplicity(collision1); | ||
| // int bin = configurableBinningDerived.getBin(std::tuple(collision1.posZ(), multiplicity)); | ||
| // float eventWeight = 1.0f / it.currentWindowNeighbours(); | ||
| if (!(collision1.sel8() && |
There was a problem hiding this comment.
We may later need to run it with a larger cfgNoMixedEvents because we skip events here
There was a problem hiding this comment.
Yes currently I am using the same set of event cuts as we applied for the same events....do you want to remove these?
| for (const auto& cftrack1 : tracks1) { | ||
| const auto& p1 = tracksP.iteratorAt(cftrack1.trackId() - tracksP.begin().globalIndex()); | ||
|
|
||
| if (p1.sign() != 1) { |
There was a problem hiding this comment.
cftrack2.track().sign() does not work?
There was a problem hiding this comment.
Yes it works but as I am using the p1 for the rest of the logic (tracks cuts, pid...) i am using this object consistently
PWGCF/TableProducer/filter2Prong.cxx
Outdated
| if (selectionPID3(p1) && selectionPID3(p2)) { | ||
| if (selectionSys(p1, false, false) && selectionSys(p2, false, false)) { |
There was a problem hiding this comment.
Move the selections before the computations (line 867)
2 participants
No description provided.