Skip to content

[PWGHF] Add centrality branch to Lc candidate tree#16343

Merged
stefanopolitano merged 2 commits into
AliceO2Group:masterfrom
GiuliaRusso02:add-centrality-lc-tree
May 29, 2026
Merged

[PWGHF] Add centrality branch to Lc candidate tree#16343
stefanopolitano merged 2 commits into
AliceO2Group:masterfrom
GiuliaRusso02:add-centrality-lc-tree

Conversation

@GiuliaRusso02
Copy link
Copy Markdown
Contributor

Add centrality (CentFT0C) branch to the candidate tables
(HfCandLcLites and HfCandLcFulls) in the Lc->pKpi tree creator.
Centrality is propagated from the collision to each candidate
via fillLiteTable and fillFullTable, consistent with how it is
already filled in the event table.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 21, 2026

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

@vkucera vkucera marked this pull request as draft May 22, 2026 10:36
@vkucera
Copy link
Copy Markdown
Collaborator

vkucera commented May 22, 2026

@GiuliaRusso02 Can you please clarify why the current code does not satisfy your needs?

@zhangbiao-phy
Copy link
Copy Markdown
Collaborator

zhangbiao-phy commented May 26, 2026

@GiuliaRusso02 Can you please clarify why the current code does not satisfy your needs?

Hi @vkucera, @GiuliaRusso02 would like to save the centrality information directly in the lite tree for convenience in ML training. For the HF derived data in OO collisions, we do not apply centrality splitting at the derived-data level. Therefore, analyzers usually run the MC or Data over a larger centrality interval, for example 0–80%. However, Giulia would like to train separate models for different centrality intervals, such as 0–20%, 20–50%, and 50–80%. In this case, it would be useful to store the centrality information in the lite tree, so that one does not need to run several trains separately for different centrality intervals.

@vkucera
Copy link
Copy Markdown
Collaborator

vkucera commented May 27, 2026

@GiuliaRusso02 Can you please clarify why the current code does not satisfy your needs?

Hi @vkucera, @GiuliaRusso02 would like to save the centrality information directly in the lite tree for convenience in ML training. For the HF derived data in OO collisions, we do not apply centrality splitting at the derived-data level. Therefore, analyzers usually run the MC or Data over a larger centrality interval, for example 0–80%. However, Giulia would like to train separate models for different centrality intervals, such as 0–20%, 20–50%, and 50–80%. In this case, it would be useful to store the centrality information in the lite tree, so that one does not need to run several trains separately for different centrality intervals.

What's the problem of getting it from the event table?

@GiuliaRusso02
Copy link
Copy Markdown
Contributor Author

@vkucera The candidate tree is used directly for ML training on Hyperloop. Having centrality already in the candidate tree avoids the need to join with the event table at training time, and allows splitting candidates by centrality interval in a single train instead of running multiple separate trains.

Moreover, I think that since the Lite candidate table does not contain a collision ID (due to unbound indices in AO2D merging, see https://aliceo2group.github.io/analysis-framework/docs/hyperloop/userdocumentation.html#8-unbound-indices-detected-in-ao2d-merging), the join with the event table at training time would actually be impossible, not just unwanted.

@vkucera
Copy link
Copy Markdown
Collaborator

vkucera commented May 28, 2026

@GiuliaRusso02 @zhangbiao-phy Thanks for the details.
Please note that, in general, if you need output tables with collision information linked to the candidates/particles, you should use the derived-data creator.
If that is too much hassle, in this case, I guess adding one column is tolerable.

@vkucera vkucera marked this pull request as ready for review May 28, 2026 12:08
@lubynets
Copy link
Copy Markdown
Contributor

@vkucera @zhangbiao-phy could you allow the CI tests for this PR (since @GiuliaRusso02 has the first contribution, the tests did not start automatically and require admins' approval)?

@stefanopolitano stefanopolitano merged commit de7b76d into AliceO2Group:master May 29, 2026
13 of 14 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.

5 participants