Skip to content

[PWGEM/Dilepton] add a small task to check MC template#14681

Merged
dsekihat merged 4 commits intoAliceO2Group:masterfrom
dsekihat:mc
Jan 29, 2026
Merged

[PWGEM/Dilepton] add a small task to check MC template#14681
dsekihat merged 4 commits intoAliceO2Group:masterfrom
dsekihat:mc

Conversation

@dsekihat
Copy link
Collaborator

@dsekihat dsekihat commented Jan 28, 2026

PWGEM/Dilepton: add a small task to check MC template

@github-actions github-actions bot added the pwgem label Jan 28, 2026
@github-actions github-actions bot changed the title PWGEM/Dilepton: add a small task to check MC template [PWGEM/Dilepton] add a small task to check MC template Jan 28, 2026
@dsekihat
Remove unused DCA related variables
d0c10e3 
Comment out unused variables related to DCA calculations.
@alibuild
Copy link
Collaborator

alibuild commented Jan 28, 2026

Error while checking build/O2Physics/o2 for d0c10e3 at 2026-01-28 20:07:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/14681-slc9_x86-64/0/PWGEM/Dilepton/Tasks/checkMCTemplate.cxx:523:5: error: 'pDCA' was not declared in this scope
ninja: build stopped: subcommand failed.

Full log here.

@dsekihat
Comment out pDCA and remove isSelectedMuon function
4351821 
Comment out pDCA calculation and remove isSelectedMuon function.
@dsekihat dsekihat enabled auto-merge (squash) January 28, 2026 21:31
alibuild
alibuild previously approved these changes Jan 28, 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.

@alibuild
Copy link
Collaborator

alibuild commented Jan 28, 2026

Error while checking build/O2Physics/o2 for 4351821 at 2026-01-28 23:09:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/14681-slc9_x86-64/0/PWGEM/Dilepton/Tasks/checkMCTemplate.cxx:473:11: error: unused variable 'dcaXY' [-Werror=unused-variable]
ninja: build stopped: subcommand failed.

Full log here.

@dsekihat dsekihat disabled auto-merge January 29, 2026 07:54
@dsekihat
Comment out dcaXY calculation
a923fb0 
Comment out the calculation of dcaXY.
@dsekihat dsekihat enabled auto-merge (squash) January 29, 2026 10:09
alibuild
alibuild approved these changes Jan 29, 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 df7e33a into AliceO2Group:master Jan 29, 2026
12 checks passed
@dsekihat dsekihat deleted the mc branch January 29, 2026 14:06
fcolamar pushed a commit to fcolamar/O2Physics that referenced this pull request Feb 4, 2026
@dsekihat
[PWGEM/Dilepton] add a small task to check MC template (AliceO2Group#…
a47e8f1 
…14681)
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

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

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

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

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

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

Assignees

No one assigned

Labels

pwgem

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dsekihat @alibuild