Skip to content

Conversation

@hfribert
Copy link
Contributor

Added a missing argument for the collision hist manager in the kink Qa task, and removed a duplicate in the trackHistManager.

@github-actions github-actions bot added the pwgcf label Oct 20, 2025
@github-actions
Copy link

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

using ConfResonanceNegDauBinning = ConfTrackBinning<PrefixResonanceNegDauBinning>;
using ConfV0PosDauBinning = ConfTrackBinning<PrefixV0PosDauBinning>;
using ConfV0NegDauBinning = ConfTrackBinning<PrefixV0NegDauBinning>;
using ConfKinkChaDauBinning = ConfTrackBinning<PrefixKinkChaDauBinning>;
Copy link
Collaborator

Choose a reason for hiding this comment

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

@hfribert hi, may I ask if this deletion was intended or accidental?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@lauraser Hi, yes I removed it because it was accidentally declared twice (lines 145 and 149).

@lauraser lauraser merged commit 9636af2 into AliceO2Group:master Oct 21, 2025
14 of 15 checks passed
ThePhDane pushed a commit to ThePhDane/O2Physics that referenced this pull request Nov 3, 2025
)

Co-authored-by: Henrik Fribert <ge35huh@nidoking.ktas.ph.tum.de>
lmattei01 pushed a commit to lmattei01/O2Physics that referenced this pull request Dec 5, 2025
)

Co-authored-by: Henrik Fribert <ge35huh@nidoking.ktas.ph.tum.de>
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