Skip to content

Conversation

@arvindkhuntia
Copy link
Contributor

Hi @nzardosh,

Could you please approve the PR once all tests are passed?

Have a nice evening.
With best regards,
Arvind

@github-actions
Copy link

github-actions bot commented Jul 21, 2025

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

@nzardosh
Copy link
Collaborator

Hi, for your new process function why not use JParticles and JMcCollisions? in a similiar way to what you have in data

@arvindkhuntia
Copy link
Contributor Author

Hi @nzardosh,
Thank you for the suggestion. Now switched to Jtables.

Have a nice day.

@nzardosh
Copy link
Collaborator

Dear @arvindkhuntia
Thanks for making the changes. However today we had this PR that was added to the JE framework which is quite relevant to your analysis
#11981
You can now get the centrality and multiplicity tables directly from the JCollision table. You can modify your task accordingly and remove the direct subscriptions to the non J tables.

Thanks,
Nima

@arvindkhuntia
Copy link
Contributor Author

arvindkhuntia commented Jul 23, 2025

Hi @nzardosh,

Thank you for adding the centrality info to Jcollisions. I am now using these info from JColl.

Cheers,
Arvind

@nzardosh nzardosh merged commit ee5398c into AliceO2Group:master Jul 24, 2025
11 of 13 checks passed
EmilGorm pushed a commit to EmilGorm/O2Physics that referenced this pull request Jul 25, 2025
Co-authored-by: Arvind Khuntia <arvind.khuntia@cern.ch>
jpxrk pushed a commit to jpxrk/O2Physics that referenced this pull request Aug 12, 2025
Co-authored-by: Arvind Khuntia <arvind.khuntia@cern.ch>
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
Co-authored-by: Arvind Khuntia <arvind.khuntia@cern.ch>
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Dec 5, 2025
Co-authored-by: Arvind Khuntia <arvind.khuntia@cern.ch>
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.

3 participants