Skip to content

[PWGCF] Update EbyE partNumFluc for MC particle - track match - #18719

Open
fsii wants to merge 1 commit into
AliceO2Group:masterfrom
fsii:dev
Open

fsii wants to merge 1 commit into
AliceO2Group:masterfrom
fsii:dev

Conversation

@fsii

@fsii fsii commented Nov 12, 2026

Copy link
Copy Markdown
Collaborator
  1. Update MC particle - track match (remove use of double-sliceBy)
  2. Delete rest C_CS and C_SV, both were added by PR15976 to avoid errors on macOS-arm (pass on Linux-x64), modified back to HIST by PR18584

Copilot AI balanced review requested due to automatic review settings November 12, 2026 16:33
@github-actions github-actions Bot added the pwgcf label Nov 12, 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.

🟢 Approval recommended

No active review issues are reported for the particle–track matching update.

No open issues 🎉

What changed in this PR

Updates Monte Carlo particle–track matching in the event-by-event particle-number fluctuation task, replacing nested slicing with explicit collision filtering.

Changes:

  • Slice ungrouped tracks by MC particle, then filter by collision for mini-table storage and analysis.
  • Count collision-matched tracks explicitly for acceptance checks and quality-assurance histograms.
  • Replace the remaining C_CS and C_SV wrappers with direct HIST calls and make minor C++ cleanups.
File Description
PWGCF/​EbyEFluctuations/​Tasks/​partNumFluc.cxx Revises MC particle–track matching and removes histogram macro wrappers.

Review effort: Balanced

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