Skip to content

[PWGHF] Xic0Omegac0: Fix bachelor Id in output table#15698

Merged
zhangbiao-phy merged 1 commit intoAliceO2Group:masterfrom
jinhyunni:dev
Apr 10, 2026
Merged

[PWGHF] Xic0Omegac0: Fix bachelor Id in output table#15698
zhangbiao-phy merged 1 commit intoAliceO2Group:masterfrom
jinhyunni:dev

Conversation

@jinhyunni
Copy link
Copy Markdown
Contributor

@jinhyunni jinhyunni commented Apr 9, 2026

Found a bug in candidate table filling.
Instead of bachelor id, the id of charm bachelor was being input.
This resulted in the candidates reconstructed with MC failing in MC matching.

Fixed file: PWGHF/TableProducer/candidateCreatorXic0Omegac0Qa.cxx

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

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

@github-actions github-actions bot changed the title Fix output table content - Wrong bachelor Id given [PWGHF] Fix output table content - Wrong bachelor Id given Apr 9, 2026
@vkucera
Copy link
Copy Markdown
Collaborator

vkucera commented Apr 9, 2026

Hi @jinhyunni , it is not clear from the title which workflow is affected. Please put at least the file name after the prefix.

@jinhyunni
Copy link
Copy Markdown
Contributor Author

Hi @vkucera, thanks for the comment.
I've updated the first comment to have the information about what file have been updated.

@jinhyunni jinhyunni changed the title [PWGHF] Fix output table content - Wrong bachelor Id given [PWGHF] Xic0Omegac0: Fix output table content - Wrong bachelor Id given Apr 9, 2026
@vkucera vkucera changed the title [PWGHF] Xic0Omegac0: Fix output table content - Wrong bachelor Id given [PWGHF] Xic0Omegac0: Fix wrong bachelor Id in output table Apr 9, 2026
@vkucera
Copy link
Copy Markdown
Collaborator

vkucera commented Apr 9, 2026

Hi @vkucera, thanks for the comment. I've updated the first comment to have the information about what file have been updated.

Thanks. I just shortened the updated title.

@vkucera vkucera changed the title [PWGHF] Xic0Omegac0: Fix wrong bachelor Id in output table [PWGHF] Xic0Omegac0: Fix bachelor Id in output table Apr 9, 2026
@jinhyunni
Copy link
Copy Markdown
Contributor Author

Hi @zhangbiao-phy, could you please review this PR when you have time?

@zhangbiao-phy
Copy link
Copy Markdown
Collaborator

Thanks @jinhyunni !

@zhangbiao-phy zhangbiao-phy merged commit 3ffd344 into AliceO2Group:master Apr 10, 2026
20 of 21 checks passed
@jinhyunni
Copy link
Copy Markdown
Contributor Author

Thanks @zhangbiao-phy for your fast response!

@jinhyunni jinhyunni deleted the dev branch April 10, 2026 05:40
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