[PWGLF] Change Generated Event selection in NucleiSpectra.cxx#15158
[PWGLF] Change Generated Event selection in NucleiSpectra.cxx#15158Cristian-Moscatelli wants to merge 2 commits intoAliceO2Group:masterfrom
Conversation
github-actions
bot
changed the title
|
I remain at disposal for any further refinement of the code, to improve the evaluation of the signal and event loss |
Cristian-Moscatelli
requested review from
HorstMa,
alibuild,
fmazzasc,
maciacco,
mpuccio,
omvazque and
skundu692
as code owners
mpuccio
left a comment
mpuccio
left a comment
There was a problem hiding this comment.
Dear @Cristian-Moscatelli,
I think what would be best here is to track the selections in a bit mask that we can store in the gen particles. See the comments below for a few more details.
| spectra.fill(HIST("hGenVtxZ"), c.posZ()); | ||
|
|
||
| // Apply the |z| < 10 cm condition | ||
| if (std::abs(c.posZ()) > 10.f) |
There was a problem hiding this comment.
You should keep track of the events you discard, also note that for the TVX definition has a different Z vertex window
| continue; | ||
|
|
||
| // Apply the TVX trigger condition | ||
| if (p.eta() > 3.5f && p.eta() < 4.9f) |
There was a problem hiding this comment.
You should request that the particle in acceptance is charged
|
|
||
| bool hasHitFT0A(false); | ||
| bool hasHitFT0C(false); | ||
| bool acceptEvent = !selectINELgt0; |
There was a problem hiding this comment.
This, in combination with line 950, makes the configurable not useful as the statement in line 950 will be always true.
|
Thank you for your comment. I will work to refine the task including your suggestions |
3 participants
I modified the selection criteria for generated events (MC collisions) in the task NucleiSpectra.cxx (PWGLF/TableProducer/Nuspex). Previously, no selections were applied to generated events. I applied the following requirements:
Additionally, I replaced the hardcoded triggerList (line 459) with a configurable parameter (line 339). The default value (fHe) is left unvaried.