Skip to content

[PWGEM/Dilepton] update muon analyses for dr vs. chi2MatchMCHMFT #14757

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 3 commits into from
Feb 3, 2026
Merged

[PWGEM/Dilepton] update muon analyses for dr vs. chi2MatchMCHMFT #14757

dsekihat merged 3 commits into from
Feb 3, 2026
+204 −79

Conversation

@dsekihat
Copy link
Collaborator

@dsekihat dsekihat commented Feb 2, 2026

PWGEM/Dilepton: update muon analyses for dr vs. chi2MatchMCHMFT

@github-actions github-actions bot added the pwgem label Feb 2, 2026
@github-actions github-actions bot changed the title PWGEM/Dilepton: update muon analyses for dr vs. chi2MatchMCHMFT [PWGEM/Dilepton] update muon analyses for dr vs. chi2MatchMCHMFT Feb 2, 2026
@alibuild alibuild mentioned this pull request Feb 2, 2026
Merged
dsekihat added a commit to dsekihat/O2Physics that referenced this pull request Feb 2, 2026
@dsekihat
Merge pull request #73 from alibuild/alibot-cleanup-14757
69d57dc 
Please consider the following formatting changes to AliceO2Group#14757
@dsekihat dsekihat enabled auto-merge (squash) February 2, 2026 18:08
alibuild
alibuild previously approved these changes Feb 2, 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 Feb 2, 2026

Error while checking build/O2Physics/o2 for 69d57dc at 2026-02-02 19:26:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/14757-slc9_x86-64/0/PWGEM/Dilepton/Tasks/matchingMFT.cxx:803:15: error: variable 'pt' set but not used [-Werror=unused-but-set-variable]
/sw/SOURCES/O2Physics/14757-slc9_x86-64/0/PWGEM/Dilepton/TableProducer/skimmerPrimaryMuon.cxx:567:40: error: unused variable 'min_dr' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/14757-slc9_x86-64/0/PWGEM/Dilepton/TableProducer/skimmerPrimaryMuon.cxx:567:56: error: unused variable 'min_distance_2d' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/14757-slc9_x86-64/0/PWGEM/Dilepton/TableProducer/skimmerPrimaryMuon.cxx:567:40: error: unused variable 'min_dr' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/14757-slc9_x86-64/0/PWGEM/Dilepton/TableProducer/skimmerPrimaryMuon.cxx:567:56: error: unused variable 'min_distance_2d' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/14757-slc9_x86-64/0/PWGEM/Dilepton/TableProducer/skimmerPrimaryMuon.cxx:567:40: error: unused variable 'min_dr' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/14757-slc9_x86-64/0/PWGEM/Dilepton/TableProducer/skimmerPrimaryMuon.cxx:567:56: error: unused variable 'min_distance_2d' [-Werror=unused-variable]
ninja: build stopped: subcommand failed.

Full log here.

@dsekihat
Clean up unused variables in skimmerPrimaryMuon.cxx
8ba1b29 
Removed unused variables min_dr and min_distance_2d from skimmerPrimaryMuon.cxx.
@dsekihat dsekihat disabled auto-merge February 2, 2026 18:50
@dsekihat dsekihat enabled auto-merge (squash) February 2, 2026 18:57
@dsekihat dsekihat disabled auto-merge February 2, 2026 18:57
alibuild
alibuild previously approved these changes Feb 2, 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
Remove unused tuple declarations in skimmerPrimaryMuon
83f5025 
Comment out unused tuple declarations for minimum distance calculations.
@dsekihat dsekihat enabled auto-merge (squash) February 2, 2026 19:29
@alibuild
Copy link
Collaborator

alibuild commented Feb 2, 2026

Error while checking build/O2Physics/o2 for f963bf6 at 2026-02-02 20:35:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/14757-slc9_x86-64/0/PWGEM/Dilepton/TableProducer/skimmerPrimaryMuon.cxx:568:40: error: unused variable 'min_dr' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/14757-slc9_x86-64/0/PWGEM/Dilepton/TableProducer/skimmerPrimaryMuon.cxx:568:56: error: unused variable 'min_distance_2d' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/14757-slc9_x86-64/0/PWGEM/Dilepton/TableProducer/skimmerPrimaryMuon.cxx:568:40: error: unused variable 'min_dr' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/14757-slc9_x86-64/0/PWGEM/Dilepton/TableProducer/skimmerPrimaryMuon.cxx:568:56: error: unused variable 'min_distance_2d' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/14757-slc9_x86-64/0/PWGEM/Dilepton/TableProducer/skimmerPrimaryMuon.cxx:568:40: error: unused variable 'min_dr' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/14757-slc9_x86-64/0/PWGEM/Dilepton/TableProducer/skimmerPrimaryMuon.cxx:568:56: error: unused variable 'min_distance_2d' [-Werror=unused-variable]
ninja: build stopped: subcommand failed.

Full log here.

@dsekihat dsekihat disabled auto-merge February 2, 2026 19:37
@alibuild
Copy link
Collaborator

alibuild commented Feb 3, 2026

Error while checking build/O2Physics/o2 for 83f5025 at 2026-02-03 06:21:

/sw/SOURCES/O2Physics/14757-slc9_x86-64/0/PWGJE/Tasks/jetFragmentation.cxx:148:8: note: variable tracking size limit exceeded with '-fvar-tracking-assignments', retrying without
  148 |   void init(InitContext&)
      |        ^~~~
[1230/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-finder-charged-qa.dir/jetFinderQA.cxx.o
[1231/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-fragmentation
[1232/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-finder-charged-qa
[1233/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-substructure-output.dir/jetSubstructureOutput.cxx.o
[1234/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-v0qa.dir/v0QA.cxx.o
[1235/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-substructure-output
[1236/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-v0qa
[1237/2891] Building CXX object PWGJE/Tasks/CMakeFiles/JetSubstructureHFPCH.dir/cmake_pch.hxx.gch
[1238/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-finder-d0-qa.dir/jetFinderD0QA.cxx.o
[1239/2891] Building CXX object PWGJE/Tasks/CMakeFiles/JetSubstructureHFPCH.dir/jetSubstructureHFPCH.cxx.o
[1240/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-track-jet-qa.dir/trackJetQA.cxx.o
[1241/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-finder-ds-qa.dir/jetFinderDsQA.cxx.o
[1242/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-finder-dplus-qa.dir/jetFinderDplusQA.cxx.o
[1243/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-finder-dstar-qa.dir/jetFinderDstarQA.cxx.o
[1244/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-substructure.dir/jetSubstructure.cxx.o
[1245/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-finder-lc-qa.dir/jetFinderLcQA.cxx.o
[1246/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-substructure
[1247/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-trigger-charged-qa.dir/jetTriggerChargedQa.cxx.o
[1248/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-finder-v0-qa.dir/jetFinderV0QA.cxx.o
[1249/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-substructure-lc-output.dir/jetSubstructureLcOutput.cxx.o
[1250/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-substructure-dstar-output.dir/jetSubstructureDstarOutput.cxx.o
[1251/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-substructure-lc-output
[1252/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-substructure-ds-output.dir/jetSubstructureDsOutput.cxx.o
[1253/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-substructure-dstar-output
[1254/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-finder-d0-qa
[1255/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-substructure-d0-output.dir/jetSubstructureD0Output.cxx.o
[1256/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-finder-dplus-qa
[1257/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-substructure-ds-output
[1258/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-finder-ds-qa
[1259/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-finder-dstar-qa
[1260/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-full-trigger-qa.dir/fullJetTriggerQATask.cxx.o
[1261/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-substructure-d0-output
[1262/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-finder-lc-qa
[1263/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-substructure-dplus-output.dir/jetSubstructureDplusOutput.cxx.o
[1264/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-trigger-charged-qa
[1265/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-finder-v0-qa
[1266/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-full-trigger-qa
[1267/2891] Linking CXX executable stage/bin/o2-analysis-je-track-jet-qa
[1268/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-substructure-b0-output.dir/jetSubstructureB0Output.cxx.o
[1269/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-substructure-dplus-output
[1270/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-substructure-bplus-output.dir/jetSubstructureBplusOutput.cxx.o
[1271/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-substructure-b0-output
[1272/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-substructure-bplus-output
[1273/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-spectra-charged.dir/jetSpectraCharged.cxx.o
[1274/2891] Building CXX object PWGJE/Tasks/CMakeFiles/O2Physicsexe-analysis-je-jet-substructure-xictoxipipi-output.dir/jetSubstructureXicToXiPiPiOutput.cxx.o
[1275/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-spectra-charged
[1276/2891] Linking CXX executable stage/bin/o2-analysis-je-jet-substructure-xictoxipipi-output

Full log here.

@dsekihat dsekihat closed this Feb 3, 2026
@dsekihat dsekihat reopened this Feb 3, 2026
@dsekihat dsekihat enabled auto-merge (squash) February 3, 2026 09:21
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 f473157 into AliceO2Group:master Feb 3, 2026
25 of 31 checks passed
@dsekihat dsekihat deleted the muon branch February 3, 2026 09:21
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