Skip to content

[Common] Refactor estimators into the same process function in centralityQa - #17849

Merged
dsekihat merged 2 commits into
AliceO2Group:masterfrom
jesgum:cent-qa
Sep 9, 2026
Merged

[Common] Refactor estimators into the same process function in centralityQa#17849
dsekihat merged 2 commits into
AliceO2Group:masterfrom
jesgum:cent-qa

Conversation

@jesgum

@jesgum jesgum commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator
  • Data and mc now uses the same flag for enabled estimators instead of it being bound to the process function
  • Fix BC de-reference when using the option useCustomCalibration

@romainschotter FYI

@github-actions github-actions Bot added the common label Sep 9, 2026
@github-actions github-actions Bot changed the title Refactor estimators into the same process function in centralityQa [Common] Refactor estimators into the same process function in centralityQa Sep 9, 2026
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

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

@jesgum
jesgum marked this pull request as ready for review September 9, 2026 16:18
@dsekihat
dsekihat merged commit 4d2113b into AliceO2Group:master Sep 9, 2026
12 checks passed
@jesgum
jesgum deleted the cent-qa branch September 10, 2026 06:38
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