Skip to content

Conversation

@ppoava
Copy link
Contributor

@ppoava ppoava commented Oct 27, 2025

The following changes were made for now:

  • Added histograms for MCH with global muon selection cuts
  • Added top-bottom seperations for MCH tracks with global muon selection cuts and for global tracks
  • Added pT dependence into 2D-histograms for above mentioned histograms, as well as for scaled MFT tracks. This is also done for top-bottom selections.
  • Modified the definition of top-bottom mixed same-sign pairs to always calculate the DCA as muon (top) minus muon (bottom)
  • Bug fix of scaled MFT momentum. Now the MCH track is properly propagated to the PV and the MCH momentum is taken from there

Work in progress

The following changes were made for now:
- Added histograms for MCH with global muon selection cuts
- Added top-bottom seperations for MCH tracks with global muon selection cuts and for global tracks
- Added pT dependence into 2D-histograms for above mentioned histograms, as well as for scaled MFT tracks. This is also done for top-bottom selections.
- Modified the definition of top-bottom mixed same-sign pairs to always calculate the DCA as muon (top) minus muon (bottom)
- Bug fix of scaled MFT momentum. Now the MCH track is properly propagated to the PV and the MCH momentum is taken from there
@github-actions
Copy link

github-actions bot commented Oct 27, 2025

O2 linter results: ❌ 69 errors, ⚠️ 97 warnings, 🔕 0 disabled

- Added histograms for all cuts and for 2D histos with pT information
- Use pT as measured by the MCH in the global histograms
- Some small other bugfixes
@ppoava ppoava changed the title [Common] Adding many new checks to muon QA [Common] Bug fixing and adding more histograms Oct 30, 2025
@ppoava ppoava marked this pull request as ready for review October 30, 2025 10:58
@ppoava ppoava requested review from a team, alibuild, ddobrigk, iarsene, jgrosseo and ktf as code owners October 30, 2025 10:58
@alibuild
Copy link
Collaborator

Error while checking build/O2Physics/o2 for 175751a at 2025-10-30 14:15:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/13564-slc9_x86-64/0/Common/Tasks/qaMuon.cxx:3187:129: error: 'dcaXPair' may be used uninitialized [-Werror=maybe-uninitialized]
/sw/SOURCES/O2Physics/13564-slc9_x86-64/0/Common/Tasks/qaMuon.cxx:3188:129: error: 'dcaYPair' may be used uninitialized [-Werror=maybe-uninitialized]
ninja: build stopped: subcommand failed.

Full log here.

@ddobrigk ddobrigk merged commit 2f2f907 into AliceO2Group:master Oct 31, 2025
12 of 13 checks passed
ThePhDane pushed a commit to ThePhDane/O2Physics that referenced this pull request Nov 3, 2025
lmattei01 pushed a commit to lmattei01/O2Physics that referenced this pull request Dec 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants