Skip to content

[PWGHF] Add flags and additional columns to the full c–d table for possible correlated-background studies - #17699

Merged
zhangbiao-phy merged 6 commits into
AliceO2Group:masterfrom
zhangbiao-phy:master
Sep 1, 2026
Merged

[PWGHF] Add flags and additional columns to the full c–d table for possible correlated-background studies#17699
zhangbiao-phy merged 6 commits into
AliceO2Group:masterfrom
zhangbiao-phy:master

Conversation

@zhangbiao-phy

@zhangbiao-phy zhangbiao-phy commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

This PR extends the c-deuteron analysis tables and selection workflow to support studies of correlated and combinatorial MC backgrounds under different mass hypotheses.

Added new columns for MC matching and candidate selection flags, and updated logic for handling candidates based on their flags.
@github-actions github-actions Bot added the pwghf PWG-HF label Sep 1, 2026
@github-actions github-actions Bot changed the title Add Flag and columns for the cd full table for possible correlated background study [PWGHF] Add Flag and columns for the cd full table for possible correlated background study Sep 1, 2026
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

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

@zhangbiao-phy
zhangbiao-phy marked this pull request as ready for review September 1, 2026 15:12
@zhangbiao-phy zhangbiao-phy changed the title [PWGHF] Add Flag and columns for the cd full table for possible correlated background study [PWGHF] Add flags and additional columns to the full c–d table for possible correlated-background studies Sep 1, 2026
@zhangbiao-phy
zhangbiao-phy enabled auto-merge (squash) September 1, 2026 15:26
alibuild
alibuild previously approved these changes Sep 1, 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

alibuild commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for bfef27f at 2026-09-01 17:47:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 8 errors and 3 warnings.'
Found 8 errors and 3 warnings.
++ [[ 8 -gt 0 ]]
++ cat /sw/BUILD/35da40cabb2c5e906380b3f071b6442ea861923d/O2Physics-code-check/errors.txt
PWGHF/D2H/Tasks/taskCd.cxx:276:15: error: member 'cmToMum' of type 'const float' is const qualified [cppcoreguidelines-avoid-const-or-ref-data-members,-warnings-as-errors]
PWGHF/TableProducer/candidateSelectorCd.cxx:74:74: error: do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead [cppcoreguidelines-pro-bounds-array-to-pointer-decay,-warnings-as-errors]
PWGHF/TableProducer/candidateSelectorCd.cxx:77:52: error: do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead [cppcoreguidelines-pro-bounds-array-to-pointer-decay,-warnings-as-errors]
PWGHF/TableProducer/candidateSelectorCd.cxx:318:34: error: variable 'pidTrackPos1Deuteron' is not initialized [cppcoreguidelines-init-variables,-warnings-as-errors]
PWGHF/TableProducer/candidateSelectorCd.cxx:319:34: error: variable 'pidTrackPos2Deuteron' is not initialized [cppcoreguidelines-init-variables,-warnings-as-errors]
PWGHF/TableProducer/candidateSelectorCd.cxx:320:34: error: variable 'pidTrackPos1Pion' is not initialized [cppcoreguidelines-init-variables,-warnings-as-errors]
PWGHF/TableProducer/candidateSelectorCd.cxx:321:34: error: variable 'pidTrackPos2Pion' is not initialized [cppcoreguidelines-init-variables,-warnings-as-errors]
PWGHF/TableProducer/candidateSelectorCd.cxx:322:34: error: variable 'pidTrackNegKaon' is not initialized [cppcoreguidelines-init-variables,-warnings-as-errors]
++ [[ 3 -gt 0 ]]
++ cat /sw/BUILD/35da40cabb2c5e906380b3f071b6442ea861923d/O2Physics-code-check/warnings.txt
PWGHF/D2H/Tasks/taskCd.cxx:274:20: warning: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays]
PWGHF/D2H/Tasks/taskCd.cxx:275:20: warning: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays]
PWGHF/TableProducer/candidateSelectorCd.cxx:107:7: warning: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays]
++ [[ 8 -gt 0 ]]
++ exit 1
--

Full log here.

@zhangbiao-phy
zhangbiao-phy enabled auto-merge (squash) September 1, 2026 16:54

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

@zhangbiao-phy
zhangbiao-phy merged commit 1dd083c into AliceO2Group:master Sep 1, 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

pwghf PWG-HF

Development

Successfully merging this pull request may close these issues.

2 participants