Skip to content

[PWGEM,Photon] Remove unnecessary registry argument from task function - #17827

Merged
mhemmer-cern merged 3 commits into
AliceO2Group:masterfrom
mhemmer-cern:pi0-flow
Sep 8, 2026
Merged

[PWGEM,Photon] Remove unnecessary registry argument from task function#17827
mhemmer-cern merged 3 commits into
AliceO2Group:masterfrom
mhemmer-cern:pi0-flow

Conversation

@mhemmer-cern

@mhemmer-cern mhemmer-cern commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator
  • Bump EMEMCClusterMCLabels to EMEMCClusterMCLabels_001 in all PWGEM tasks where the old version was still wrongly used.
  • Fix PDG value check for electrons. This used std::abs() which was also true for positrons. Now it's explicitly only electrons.
  • Remove unnecessary registry argument from emcal-mc-task function

@github-actions github-actions Bot added pwgem photon Photon and neutral meson PAG label labels Sep 8, 2026
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

O2 linter results: ❌ 77 errors, ⚠️ 95 warnings, 🔕 3 disabled

@mhemmer-cern
mhemmer-cern enabled auto-merge (squash) September 8, 2026 10:19

@alibuild alibuild 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.

Auto-approving on behalf of @mhemmer-cern.

@mhemmer-cern
mhemmer-cern merged commit fa1066a into AliceO2Group:master Sep 8, 2026
16 of 17 checks passed
@mhemmer-cern
mhemmer-cern deleted the pi0-flow branch September 8, 2026 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

photon Photon and neutral meson PAG label pwgem

Development

Successfully merging this pull request may close these issues.

2 participants