Skip to content

[PWGHF] Fix column mismatch, restore momentum branches, add DCAFitter opts. for Xic0ToXiPi DCA Filter analysis #13811

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
zhangbiao-phy merged 3 commits into from
Nov 15, 2025
Merged

[PWGHF] Fix column mismatch, restore momentum branches, add DCAFitter opts. for Xic0ToXiPi DCA Filter analysis #13811

zhangbiao-phy merged 3 commits into from
Nov 15, 2025

Conversation

@Tao-Fang
Copy link
Contributor

@Tao-Fang Tao-Fang commented Nov 13, 2025

No description provided.

@github-actions github-actions bot added the pwghf PWG-HF label Nov 13, 2025
@github-actions github-actions bot changed the title Add momentum components for charm baryon and cascades [PWGHF] Add momentum components for charm baryon and cascades Nov 13, 2025
@github-actions
Copy link

github-actions bot commented Nov 13, 2025 •
edited
Loading

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

@Tao-Fang Tao-Fang marked this pull request as ready for review November 13, 2025 08:37
@Tao-Fang Tao-Fang changed the title [PWGHF] Add momentum components for charm baryon and cascades [PWGHF] Add momentum components for charm baryon and cascades to Xic0ToXiPi DCA Filter analysis Nov 13, 2025
@vkucera
Copy link
Collaborator

vkucera commented Nov 13, 2025

Is this a bug fix? Because the columns that you added seem different from the quantities that you added to fill.

@Tao-Fang
Copy link
Contributor Author

Tao-Fang commented Nov 13, 2025

Is this a bug fix? Because the columns that you added seem different from the quantities that you added to fill.
Yes, this is a fix for the mismatch between the two columns, and adding back the momentum components.

@vkucera
Copy link
Collaborator

vkucera commented Nov 13, 2025

Is this a bug fix? Because the columns that you added seem different from the quantities that you added to fill.
Yes, this is a fix for the mismatch between the two columns, and adding back the momentum components.

So please say it explicitly in the PR title.

@Tao-Fang Tao-Fang changed the title [PWGHF] Add momentum components for charm baryon and cascades to Xic0ToXiPi DCA Filter analysis [PWGHF] Fix mismatch of the columns, and adding back the momentum components for Xic0ToXiPi DCA Filter analysis Nov 13, 2025
@Tao-Fang Tao-Fang changed the title [PWGHF] Fix mismatch of the columns, and adding back the momentum components for Xic0ToXiPi DCA Filter analysis [PWGHF] Fix mismatch of the columns, and adding back the momentum components in tree treeCreat and add DCAFitter options to process conditions in task for Xic0ToXiPi DCA Filter analysis Nov 14, 2025
@zhangbiao-phy zhangbiao-phy merged commit a916c84 into AliceO2Group:master Nov 15, 2025
14 checks passed
@vkucera
Copy link
Collaborator

vkucera commented Nov 15, 2025 •
edited
Loading

@Tao-Fang Please use more concise PR titles and avoid typos. It is encouraged to provide more details in the PR description (below the title).

@Tao-Fang Tao-Fang changed the title [PWGHF] Fix mismatch of the columns, and adding back the momentum components in tree treeCreat and add DCAFitter options to process conditions in task for Xic0ToXiPi DCA Filter analysis [PWGHF]: Fix column mismatch, restore momentum branches, add DCAFitter opts. for Xic0ToXiPi DCA Filter analysis Nov 15, 2025
@github-actions github-actions bot changed the title [PWGHF]: Fix column mismatch, restore momentum branches, add DCAFitter opts. for Xic0ToXiPi DCA Filter analysis [PWGHF] Fix column mismatch, restore momentum branches, add DCAFitter opts. for Xic0ToXiPi DCA Filter analysis Nov 15, 2025
lmattei01 pushed a commit to lmattei01/O2Physics that referenced this pull request Dec 5, 2025
@Tao-Fang
[PWGHF] Fix mismatch of the columns, and adding back the momentum com…
deb7624 
…ponents in tree treeCreat and add DCAFitter options to process conditions in task for Xic0ToXiPi DCA Filter analysis (AliceO2Group#13811)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zhangbiao-phy zhangbiao-phy zhangbiao-phy approved these changes

@alibuild alibuild Awaiting requested review from alibuild alibuild is a code owner

@vkucera vkucera Awaiting requested review from vkucera vkucera is a code owner

@fcolamar fcolamar Awaiting requested review from fcolamar fcolamar is a code owner

@fgrosa fgrosa Awaiting requested review from fgrosa fgrosa is a code owner

@fcatalan92 fcatalan92 Awaiting requested review from fcatalan92 fcatalan92 is a code owner

@mfaggin mfaggin Awaiting requested review from mfaggin mfaggin is a code owner

@mmazzilli mmazzilli Awaiting requested review from mmazzilli mmazzilli is a code owner

@deepathoms deepathoms Awaiting requested review from deepathoms deepathoms is a code owner

@NicoleBastid NicoleBastid Awaiting requested review from NicoleBastid NicoleBastid is a code owner

@hahassan7 hahassan7 Awaiting requested review from hahassan7 hahassan7 is a code owner

@jpxrk jpxrk Awaiting requested review from jpxrk jpxrk is a code owner

@apalasciano apalasciano Awaiting requested review from apalasciano apalasciano is a code owner

@gluparel gluparel Awaiting requested review from gluparel gluparel is a code owner

@stefanopolitano stefanopolitano Awaiting requested review from stefanopolitano stefanopolitano is a code owner

Assignees

No one assigned

Labels

pwghf PWG-HF

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Tao-Fang @vkucera @zhangbiao-phy