-
Notifications
You must be signed in to change notification settings - Fork 613
[PWGLF] Add task to produce binned information about V0s/cascades #12212
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.
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
Conversation
|
O2 linter results: ❌ 62 errors, |
Please consider the following formatting changes to AliceO2Group#12212
alibuild
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Auto-approving on behalf of @romainschotter.
|
Thank you for your code, @romainschotter As discussed on Mattermost and in person, I have some comments on configuration and implementation and would like to share the reasoning behind the strategy I’ve successfully applied for multidimensional group-by statistics extraction since Run 1 (and which contributed to the development of THn in ROOT about 15 years ago).
Configurable Binning
Why Functional Transformations Matter
Dealing with Sparse Bins
This methodology allowed me to generate performance maps for Run 1, Run 2, and early Run 3. The implementation is currently in AliRoot, but I’m considering porting it to O2, as mixing O2 and AliRoot in the same workflow caused some issues recently - I assume then name clashes for some code - if they used the same variable names in AliRoot and O2. |
…iceO2Group#12212) Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
…iceO2Group#12212) Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
…iceO2Group#12212) Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
Add task running over strangeness derived data and producing binned information about V0s/cascades
@miranov25 for your information