Skip to content

[PWGHF] feat(taskLcToK0sP): add THnSparse with ML scores, align with taskLc patterns#16492

Open
dongfangqingxin wants to merge 10 commits into
AliceO2Group:masterfrom
dongfangqingxin:master
Open

[PWGHF] feat(taskLcToK0sP): add THnSparse with ML scores, align with taskLc patterns#16492
dongfangqingxin wants to merge 10 commits into
AliceO2Group:masterfrom
dongfangqingxin:master

Conversation

@dongfangqingxin
Copy link
Copy Markdown
Contributor

Add THnSparse support for Λc→K0sP analysis with BDT scores (Data, MC Rec, MC Gen). Add process functions for standard/ML with FT0C/FT0M centrality estimators. Refactor code structure to align with taskLc.cxx."

@github-actions github-actions Bot added the pwghf PWG-HF label Jun 1, 2026
@github-actions github-actions Bot changed the title feat(taskLcToK0sP): add THnSparse with ML scores, align with taskLc patterns [PWGHF] feat(taskLcToK0sP): add THnSparse with ML scores, align with taskLc patterns Jun 1, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 1, 2026

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

dongfangqingxin added a commit to dongfangqingxin/O2Physics that referenced this pull request Jun 1, 2026
Please consider the following formatting changes to AliceO2Group#16492
@vkucera vkucera marked this pull request as draft June 1, 2026 21:31
@vkucera
Copy link
Copy Markdown
Collaborator

vkucera commented Jun 1, 2026

Fix the reported issues.

…atterns

Add THnSparse support for Λc→K0sP analysis with BDT scores (Data,
MC Rec, MC Gen). Add process functions for standard/ML with FT0C/FT0M
centrality estimators. Refactor code structure to align with taskLc.cxx.
@vkucera
Copy link
Copy Markdown
Collaborator

vkucera commented Jun 2, 2026

@dongfangqingxin Thanks for the fixes. You have overwritten the formatting changes four times. Please learn how to update your local branch and/or how to format your local changes automatically to avoid it.

@dongfangqingxin
Copy link
Copy Markdown
Contributor Author

@dongfangqingxin Thanks for the fixes. You have overwritten the formatting changes four times. Please learn how to update your local branch and/or how to format your local changes automatically to avoid it.

Hi @vkucera Thanks for the heads-up and sorry for the back-and-forth. I've installed clang-format locally now, will make sure to run it before pushing next time.

@vkucera
Copy link
Copy Markdown
Collaborator

vkucera commented Jun 2, 2026

@dongfangqingxin Thanks for the fixes. You have overwritten the formatting changes four times. Please learn how to update your local branch and/or how to format your local changes automatically to avoid it.

Hi @vkucera Thanks for the heads-up and sorry for the back-and-forth. I've installed clang-format locally now, will make sure to run it before pushing next time.

You don't need to install and run it separately, if you're using the pre-commit hooks.

@dongfangqingxin
Copy link
Copy Markdown
Contributor Author

@dongfangqingxin Thanks for the fixes. You have overwritten the formatting changes four times. Please learn how to update your local branch and/or how to format your local changes automatically to avoid it.

Hi @vkucera Thanks for the heads-up and sorry for the back-and-forth. I've installed clang-format locally now, will make sure to run it before pushing next time.

You don't need to install and run it separately, if you're using the pre-commit hooks.

Ah, I see — thanks for the tip! I hadn't set up the pre-commit hooks. I'll activate them now to avoid this in the future.

@vkucera vkucera marked this pull request as ready for review June 2, 2026 07:27
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