Skip to content

[PWGDQ] Improvements tableReader_withAssoc and dqEfficiency_withAssoc, and simplifying FillPairVertexing - #17999

Open
mcoquet642 wants to merge 9 commits into
AliceO2Group:masterfrom
mcoquet642:dq-tr-muon-fixes
Open

mcoquet642 wants to merge 9 commits into
AliceO2Group:masterfrom
mcoquet642:dq-tr-muon-fixes

Conversation

@mcoquet642

Copy link
Copy Markdown
Collaborator
  • Same fix as in PR#17893 to enable best muon skimming when not using ML model
  • tableReader_withAssoc : same-event-pairing : filling histograms only when fConfigQA is true
  • dqEfficiency/tableReader_withAssoc : muon-selection : Removing unnecessary fetching of GRPMag from ccdb
  • dqEfficiency/tableReader_withAssoc : same-event-pairing : Removing unnecessary fetching of geometry from ccdb
  • dqEfficiency/tableReader_withAssoc : same-event-pairing : Removing deprecated option cfgCorrFullGeo
  • dqEfficiency/tableReader_withAssoc : same-event-pairing : Removing manual setting of the magnetic field
  • dqEfficiency/tableReader_withAssoc : same-event-pairing : muon vertexing is now enabled only for the process
    processMuonOnlyVertexingSkimmed. For other processes, vertexing is disabled
  • VarManager/dqEfficiency/tableReader_withAssoc : removing fPropToPCA option, which should always be true
  • VarManager : Simplifying the code for FillPairVertexing : removing unnecessary calculations and improving readability

@github-actions github-actions Bot added the pwgdq label Sep 21, 2026
@github-actions github-actions Bot changed the title [PWGDQ] Improvements tableReader_withAssoc and dqEfficiency_withAssoc, and simplifying FillPairVertexing [PWGDQ] Improvements tableReader_withAssoc and dqEfficiency_withAssoc, and simplifying FillPairVertexing Sep 21, 2026
@github-actions

Copy link
Copy Markdown

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

Please consider the following formatting changes to AliceO2Group#17999
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.

2 participants