Skip to content

[PWGEM] Add option to save Dalitz and two-gamma events - #18363

Merged
jokonig merged 1 commit into
AliceO2Group:masterfrom
jokonig:master
Oct 20, 2026
Merged

[PWGEM] Add option to save Dalitz and two-gamma events#18363
jokonig merged 1 commit into
AliceO2Group:masterfrom
jokonig:master

Conversation

@jokonig

@jokonig jokonig commented Oct 20, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

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

github-actions Bot commented Oct 20, 2026

Copy link
Copy Markdown

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

@jokonig
jokonig marked this pull request as ready for review October 20, 2026 11:39
@alibuild

alibuild commented Oct 20, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for 84344f1 at 2026-10-20 13:57:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGEM/PhotonMeson/TableProducer/skimmerDalitzEvents.cxx:88:59: error: suggest parentheses around '&&' within '||' [-Werror=parentheses]
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGEM/PhotonMeson/TableProducer/skimmerDalitzEvents.cxx:88:59: error: suggest parentheses around '&&' within '||' [-Werror=parentheses]
ninja: build stopped: subcommand failed.

Full log here.

@alibuild

alibuild commented Oct 20, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 84344f1 at 2026-10-20 14:06:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGEM/PhotonMeson/TableProducer/skimmerDalitzEvents.cxx:88:59: error: suggest parentheses around '&&' within '||' [-Werror=parentheses]
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGEM/PhotonMeson/TableProducer/skimmerDalitzEvents.cxx:88:59: error: suggest parentheses around '&&' within '||' [-Werror=parentheses]
ninja: build stopped: subcommand failed.

Full log here.

@jokonig
jokonig force-pushed the master branch 2 times, most recently from fb2d716 to 537c17f Compare October 20, 2026 12:19
@jokonig
jokonig enabled auto-merge (squash) October 20, 2026 12:48
alibuild
alibuild approved these changes Oct 20, 2026
View reviewed changes

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

@jokonig
jokonig merged commit 73d3c4d into AliceO2Group:master Oct 20, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alibuild alibuild alibuild approved these changes
@mhemmer-cern mhemmer-cern Awaiting requested review from mhemmer-cern mhemmer-cern is a code owner
@dsekihat dsekihat Awaiting requested review from dsekihat dsekihat is a code owner
@hscheid hscheid Awaiting requested review from hscheid hscheid is a code owner
@amaringarcia amaringarcia Awaiting requested review from amaringarcia amaringarcia is a code owner
@reygers reygers Awaiting requested review from reygers reygers is a code owner

Assignees

No one assigned

Labels

photon Photon and neutral meson PAG label pwgem

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jokonig @alibuild