[PWGHF] Replace UPC Lc THnSparse with TTree#15786
[PWGHF] Replace UPC Lc THnSparse with TTree#15786Rrantu wants to merge 8 commits intoAliceO2Group:masterfrom
Conversation
|
O2 linter results: ❌ 0 errors, |
|
@Rrantu Columns and tables are parts of the data model. They don't belong in the utility headers. |
|
Hi @vkucera, thanks for the comment, where would you recommend moving the columns and tables to? Would it be acceptable to define them directly in the task? |
Yes, if the tables are not used anywhere else, please put them in the task. |
Thanks! I‘ve moved them to the task file |
|
@Rrantu A conceptual question: Why can you not use the THnSparse? |
Thanks for the question! For large statistics, merging THnSparse objects becomes very heavy, making it difficult to obtain merged AnalysisResults.root. I also need to repeatedly adjust selections on multiple axes during the offline analysis. Projections from high-dimensional THnSparse are not very flexible. |
|
Dear @Rrantu, @vkucera, all codeowners, sorry for interfering, |
No, they are not. |
|
Hi @lubynets, Thanks for the question. In this case, the tables are only used within this task, and the trees are produced specifically for this analysis. Also, there are other tasks that produce trees in a similar way. |
This PR introduces a transition from THnSparse to TTree in the UPC Lc analysis.
Main changes: