Skip to content

[PWGCF] Fix TOF selection logic of daughter tracks in femto framework - #18202

Merged
ariedel-cern merged 4 commits into
AliceO2Group:masterfrom
ariedel-cern:master
Oct 7, 2026
Merged

ariedel-cern merged 4 commits into
AliceO2Group:masterfrom
ariedel-cern:master

Conversation

@ariedel-cern

Copy link
Copy Markdown
Collaborator

Fix TOF selection logic of daughter tracks

  • Fixed requireTof=true + keepTracksWithoutTof=false not actually rejecting daughters without a TOF signal (the mandatory cut was silently never evaluated).
  • Fixed skipMostPermissiveBit being hardcoded true for TOF cuts, which dropped the loosest-threshold bit even when requireTof=false (optional cut). Now tied to requireTof.
  • Clarified Configurable descriptions for requireTof/keepTracksWithoutTof.

@github-actions github-actions Bot added the pwgcf label Oct 7, 2026
@github-actions

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown

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

Please consider the following formatting changes to AliceO2Group#18202
@ariedel-cern
ariedel-cern marked this pull request as ready for review October 7, 2026 14:10
@ariedel-cern
ariedel-cern enabled auto-merge (squash) October 7, 2026 14:31
alibuild
alibuild previously approved these changes Oct 7, 2026

@alibuild alibuild left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Auto-approving on behalf of @ariedel-cern.

@alibuild alibuild left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Auto-approving on behalf of @ariedel-cern.

@ariedel-cern
ariedel-cern merged commit b286a24 into AliceO2Group:master Oct 7, 2026
14 of 15 checks passed
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