Skip to content

[PWGHF] Refactor mixed event handling for track selection in Charm Track femto task#17191

Open
zhangbiao-phy wants to merge 2 commits into
AliceO2Group:masterfrom
zhangbiao-phy:master
Open

[PWGHF] Refactor mixed event handling for track selection in Charm Track femto task#17191
zhangbiao-phy wants to merge 2 commits into
AliceO2Group:masterfrom
zhangbiao-phy:master

Conversation

@zhangbiao-phy

@zhangbiao-phy zhangbiao-phy commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

Updated mixed event handling to use selected partition for track 1 based on PDG code. This change applies to multiple decay channels and ensures correct partitioning for mixed events. The effect only for the MC mixed-event

Updated mixed event handling to use selected partition for track 1 based on PDG code. This change applies to multiple decay channels and ensures correct partitioning for mixed events.
@github-actions github-actions Bot added the pwghf PWG-HF label Jul 24, 2026
@github-actions github-actions Bot changed the title Refactor mixed event handling for track selection in Charm Track femto task [PWGHF] Refactor mixed event handling for track selection in Charm Track femto task Jul 24, 2026
@github-actions

Copy link
Copy Markdown

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

alibuild
alibuild previously approved these changes Jul 24, 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 @zhangbiao-phy.

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for b6de1a3 at 2026-07-24 15:58:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 8 errors and 0 warnings.'
Found 8 errors and 0 warnings.
++ [[ 8 -gt 0 ]]
++ cat /sw/BUILD/0790eeeea0985c198490546113f014df9a356ccc/O2Physics-code-check/errors.txt
PWGHF/HFC/Tasks/taskCharmHadronsTrackFemtoDream.cxx:385:9: error: do not use 'else' after 'return' [readability-else-after-return,-warnings-as-errors]
PWGHF/HFC/Tasks/taskCharmHadronsTrackFemtoDream.cxx:401:9: error: do not use 'else' after 'return' [readability-else-after-return,-warnings-as-errors]
PWGHF/HFC/Tasks/taskCharmHadronsTrackFemtoDream.cxx:624:132: error: the parameter 'policy' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]
PWGHF/HFC/Tasks/taskCharmHadronsTrackFemtoDream.cxx:915:9: error: do not use 'else' after 'continue' [readability-else-after-return,-warnings-as-errors]
PWGHF/HFC/Tasks/taskCharmHadronsTrackFemtoDream.cxx:961:9: error: do not use 'else' after 'continue' [readability-else-after-return,-warnings-as-errors]
PWGHF/HFC/Tasks/taskCharmHadronsTrackFemtoDream.cxx:1006:9: error: do not use 'else' after 'continue' [readability-else-after-return,-warnings-as-errors]
PWGHF/HFC/Tasks/taskCharmHadronsTrackFemtoDream.cxx:1051:9: error: do not use 'else' after 'continue' [readability-else-after-return,-warnings-as-errors]
PWGHF/HFC/Tasks/taskCharmHadronsTrackFemtoDream.cxx:1094:9: error: do not use 'else' after 'continue' [readability-else-after-return,-warnings-as-errors]
++ [[ 0 -gt 0 ]]
++ [[ 8 -gt 0 ]]
++ exit 1
--

Full log here.

@vkucera
vkucera disabled auto-merge July 24, 2026 14:15
@vkucera

vkucera commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

@zhangbiao-phy Please fix the errors.

@zhangbiao-phy

Copy link
Copy Markdown
Collaborator Author

@zhangbiao-phy Please fix the errors.

Thanks for reminding me!

@zhangbiao-phy
zhangbiao-phy enabled auto-merge (squash) July 24, 2026 17:57

@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 @zhangbiao-phy.

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.

3 participants