Skip to content

[PWGLF] Fix PDG-related bug#15669

Open
PaolaVT wants to merge 47 commits intoAliceO2Group:masterfrom
PaolaVT:master
Open

[PWGLF] Fix PDG-related bug#15669
PaolaVT wants to merge 47 commits intoAliceO2Group:masterfrom
PaolaVT:master

Conversation

@PaolaVT
Copy link
Copy Markdown
Contributor

@PaolaVT PaolaVT commented Apr 7, 2026

Fixed a bug in the PDG code handling that caused empty proton histograms. Additional changes were made to the analysis flow to ensure correct processing.

PaolaVT and others added 30 commits July 29, 2025 13:29
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2026

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

@github-actions github-actions bot changed the title Fix PDG-related bug [PWGLF] Fix PDG-related bug Apr 7, 2026
Comment on lines +158 to +160
static constexpr int PDGPion = PDG_t::kPiPlus; // Changed from kPiPlus
static constexpr int PDGKaon = PDG_t::kKPlus; // Changed from kKPlus
static constexpr int PDGProton = PDG_t::kProton; // Changed from kProton
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.

Why don't you use the constants directly?

Comment on lines 151 to 156
@@ -174,11 +155,10 @@ struct MultiplicityPt {
kNSpecies = 3
};
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.

Don't use the k prefix.

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