-
Notifications
You must be signed in to change notification settings - Fork 614
[Common] Unified, simple, fully self-configuring mult/cent provider #11495
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: ❌ 32 errors, |
|
Tagging @romainschotter - whenever you have a moment, please help me with testing Tagging @iarsene - this should provide a measurable reduction in memory footprint but you should only use this once it is working and has been confirmed to lead to the same output as the legacy tasks Some further features to be added:
|
|
@fjonasALICE @nstrangm this PR will provide a unified mult/cent task that will include BC-level centrality values for FT0 signals and, while at it, it will simplify the task, using less memory and less CPU. I will finish some testing soon and then move the PR to review, but even then, since this is intended to be a replacement for multiplicityTable and centralityTable, we'll have to commission it a bit. Hope this helps! |
…e + remove superfluous table enabling checks
Fix multCentTable + multModule
… + add filling of tracklets table in Run 3 with dummy values
Further fix multCentTable
Add multiplicity + centrality for Run 2 converted data
Please consider the following formatting changes
Add SPD clusters L0&L1 multiplicities
|
We're still missing the autodetect at |
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 @ddobrigk.
…liceO2Group#11495) Co-authored-by: David Dobrigkeit Chinellato <david.dobrigkeit.chinellato.cern.ch> Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com> Co-authored-by: romainschotter <romain.schotter@gmail.com> Co-authored-by: SCHOTTER Romain <47983209+romainschotter@users.noreply.github.com>
…liceO2Group#11495) Co-authored-by: David Dobrigkeit Chinellato <david.dobrigkeit.chinellato.cern.ch> Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com> Co-authored-by: romainschotter <romain.schotter@gmail.com> Co-authored-by: SCHOTTER Romain <47983209+romainschotter@users.noreply.github.com>
…liceO2Group#11495) Co-authored-by: David Dobrigkeit Chinellato <david.dobrigkeit.chinellato.cern.ch> Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com> Co-authored-by: romainschotter <romain.schotter@gmail.com> Co-authored-by: SCHOTTER Romain <47983209+romainschotter@users.noreply.github.com>
…liceO2Group#11495) Co-authored-by: David Dobrigkeit Chinellato <david.dobrigkeit.chinellato.cern.ch> Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com> Co-authored-by: romainschotter <romain.schotter@gmail.com> Co-authored-by: SCHOTTER Romain <47983209+romainschotter@users.noreply.github.com>
…liceO2Group#11495) Co-authored-by: David Dobrigkeit Chinellato <david.dobrigkeit.chinellato.cern.ch> Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com> Co-authored-by: romainschotter <romain.schotter@gmail.com> Co-authored-by: SCHOTTER Romain <47983209+romainschotter@users.noreply.github.com>



strangenessbuilder. Track loops / slices replaced by single operations for faster operation in all processing paths.defineDataProcessingautodetect, but will follow in separate PR.