Skip to content

[Trigger,PWGEM] update globalDimuonFilter.cxx #14760

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
dsekihat merged 2 commits into from
Feb 3, 2026
Merged

[Trigger,PWGEM] update globalDimuonFilter.cxx #14760

dsekihat merged 2 commits into from
Feb 3, 2026
+117 −33

Conversation

@dsekihat
Copy link
Collaborator

@dsekihat dsekihat commented Feb 2, 2026

Trigger,PWGEM: update globalDimuonFilter.cxx

@alibuild
Copy link
Collaborator

alibuild commented Feb 2, 2026

Error while checking build/O2Physics/o2 for 0573054 at 2026-02-03 00:01:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/14760-slc9_x86-64/0/EventFiltering/PWGEM/globalDimuonFilter.cxx:302:14: error: variable 'mchtrack' set but not used [-Werror=unused-but-set-variable]
/sw/SOURCES/O2Physics/14760-slc9_x86-64/0/EventFiltering/PWGEM/globalDimuonFilter.cxx:303:14: error: variable 'mfttrack' set but not used [-Werror=unused-but-set-variable]
ninja: build stopped: subcommand failed.

Full log here.

@dsekihat
Comment out track matching lines in globalDimuonFilter
9d13f0e 
Comment out MCH and MFT track matching lines for clarity.
@github-actions github-actions bot changed the title Trigger,PWGEM: update globalDimuonFilter.cxx [Trigger,PWGEM] update globalDimuonFilter.cxx Feb 3, 2026
@dsekihat dsekihat enabled auto-merge (squash) February 3, 2026 10:07
alibuild
alibuild approved these changes Feb 3, 2026
View reviewed changes
Copy link
Collaborator

@alibuild alibuild left a comment

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

@dsekihat dsekihat merged commit 9a174d2 into AliceO2Group:master Feb 3, 2026
12 checks passed
@dsekihat dsekihat deleted the cefp branch February 3, 2026 10:43
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

@mpuccio mpuccio Awaiting requested review from mpuccio mpuccio is a code owner

@lietava lietava Awaiting requested review from lietava lietava is a code owner

@fgrosa fgrosa Awaiting requested review from fgrosa fgrosa is a code owner

@ariedel-cern ariedel-cern Awaiting requested review from ariedel-cern ariedel-cern is a code owner

Assignees

No one assigned

Labels

pwgem trigger

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dsekihat @alibuild