Skip to content

[PWGLF] Fixed feed-down and added multiplicity-differential selection - #17639

Merged
romainschotter merged 1 commit into
AliceO2Group:masterfrom
sprasadphy:master
Aug 27, 2026
Merged

[PWGLF] Fixed feed-down and added multiplicity-differential selection#17639
romainschotter merged 1 commit into
AliceO2Group:masterfrom
sprasadphy:master

Conversation

@sprasadphy

Copy link
Copy Markdown
Contributor

Following changes have been made:

  1. For Lambda feeddown: MC-matched candidates now also require isPhysicalPrimary(), so the efficiency numerator is consistent with the generated denominator. Non-primary candidates are filled into new hMassFeedDown*pTFlat, hFeedDown*PtVsMotherPt and hFeedDown*MotherPdg histograms.

  2. Added aod::CentFT0Ms for the multiplicity-differential analysis.

  3. The configurables are now organised into ConfigurableGroups

  4. Fixed the code-check issues: removed the unused PIDResponseTOF include, initialised the PDG service and loop variables in estimateFlattenicityFV0MC, and replaced the C-style arrays with std::array. clang-tidy is now clean for this file.

@github-actions

Copy link
Copy Markdown

O2 linter results: ❌ 0 errors, ⚠️ 46 warnings, 🔕 1 disabled

@romainschotter romainschotter left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @sprasadphy !
I had a look at the PR and it looks fine!
I just would like to draw your attention to fact that, because some configurables have been grouped into configurable groups, the configurables will be reset to their default values when resyncing the wagon in Hyperloop (Hyperloop sees it as you removed the configurable and created a new one). This means that you will have to change each configurable to its previous value by hand (which can be a long and painful operation)
I just want to be sure that you are aware of it before I merge these changes :-)
Thank you very much!

@sprasadphy

Copy link
Copy Markdown
Contributor Author

Hi @romainschotter Thank you for your concern and message. I previously did not plan on grouping them but I was reaching the limit (I think it is 100). I am well aware that I will have to change each configurable to its previous value by hand.

@romainschotter
romainschotter enabled auto-merge (squash) August 27, 2026 13:41
@romainschotter
romainschotter merged commit c9a1414 into AliceO2Group:master Aug 27, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants