Skip to content

[PWGHF] fixes to D+ inv mass THnSparse for PV contributors#16585

Merged
stefanopolitano merged 2 commits into
AliceO2Group:masterfrom
fjonasALICE:fjonas/dplus-pv-contributors
Jun 9, 2026
Merged

[PWGHF] fixes to D+ inv mass THnSparse for PV contributors#16585
stefanopolitano merged 2 commits into
AliceO2Group:masterfrom
fjonasALICE:fjonas/dplus-pv-contributors

Conversation

@fjonasALICE

Copy link
Copy Markdown
Collaborator
  • added proper handling of PV contributor axis for invMass THnSparse
  • In particular, fixed a critical bug which occured when running without storing centrality, without occupancy but with PV contributors. In this case the lines:
     if (storePvContributors) {
        axes.push_back(thnAxisPvContributors);
      }

were missing, so the ThNSparse had not enough axes. This meant that in the filling, the occupancy was treated as a weight. By default, number of pv contributors was -1, so every axis got filled with weight -1 for all quantities when running in this mode.

Changes were validated in local running on HF derived data.

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

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

@stefanopolitano stefanopolitano merged commit 44e5ef2 into AliceO2Group:master Jun 9, 2026
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pwghf PWG-HF

Development

Successfully merging this pull request may close these issues.

2 participants