Skip to content

[PWGCF] Update DCA check#17205

Merged
shouqiye merged 1 commit into
AliceO2Group:masterfrom
fsii:dev
Jul 26, 2026
Merged

[PWGCF] Update DCA check#17205
shouqiye merged 1 commit into
AliceO2Group:masterfrom
fsii:dev

Conversation

@fsii

@fsii fsii commented Jul 26, 2026

Copy link
Copy Markdown
Contributor
  1. Add check of centrality dependence of DCA
  2. Update default values of configurables
  3. Enable cut for MC z-vtx
  4. New interpolation of TH3

@fsii
fsii requested a review from alibuild as a code owner July 26, 2026 14:21
Copilot AI review requested due to automatic review settings July 26, 2026 14:21
@github-actions github-actions Bot added the pwgcf label Jul 26, 2026
@github-actions

Copy link
Copy Markdown

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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the EbyE particle-number fluctuation task to support a centrality-dependent DCA QA, refreshes several configurable defaults, enables an MC z-vertex selection cut via a configurable threshold, and replaces ROOT’s TH3::Interpolate usage with a custom interpolation routine for PID shift maps.

Changes:

  • Switch DCA QA from (pT, η, φ) dependence to (centrality, pT, η) using a new centrality calibration axis.
  • Introduce centralityCalibration (optionally distinct from the analysis centrality definition) and use NTPV centrality for QA/calibration by default.
  • Update MC handling/configurables (MC z-vtx cut threshold, PROCESS_SWITCH macros) and implement custom TH3 interpolation for NSigma shift recalibration.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread PWGCF/EbyEFluctuations/Tasks/partNumFluc.cxx
@shouqiye
shouqiye merged commit fd9fe7a into AliceO2Group:master Jul 26, 2026
14 of 15 checks passed
@fsii
fsii deleted the dev branch July 26, 2026 14:57
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