Skip to content

Conversation

@Luzhiyongg
Copy link
Contributor

@Luzhiyongg Luzhiyongg commented Jul 18, 2025

add pt container for pt correlations and vn-pt correlations

@github-actions
Copy link

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

@github-actions github-actions bot changed the title add pt container and remove some old code due to the limit num of o2 objects [PWGCF] add pt container and remove some old code due to the limit num of o2 objects Jul 18, 2025
Copy link
Collaborator

@victor-gonzalez victor-gonzalez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the limit is in the number of Configurables you can always use the ConfigurableGroups
If the limit is in the number of histograms you can always use more than one HistogramRegistry. In one of my QA task I remember using 30 so it is perfectly safe

@victor-gonzalez victor-gonzalez enabled auto-merge (squash) July 18, 2025 16:52
@victor-gonzalez victor-gonzalez merged commit e4127a8 into AliceO2Group:master Jul 22, 2025
15 of 16 checks passed
vojmach pushed a commit to vojmach/O2Physics that referenced this pull request Jul 23, 2025
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Dec 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants