Skip to content

Conversation

@arvindkhuntia
Copy link
Contributor

Dear @nzardosh,

This PR just fixed the 3D histogram filling. Please approve the PR once tests are done.

Have a nice day.

@github-actions
Copy link

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

@nzardosh nzardosh enabled auto-merge (squash) July 30, 2025 11:43
@vkucera
Copy link
Collaborator

vkucera commented Jul 30, 2025

Hi @arvindkhuntia , you have plenty of hypotenuse calculations. Consider using existing features like std::hypot or ‎‎‎RecoDecay::sqrtSumOfSquares.

@vkucera
Copy link
Collaborator

vkucera commented Jul 30, 2025

Also consider fixing the errors.

@nzardosh nzardosh merged commit af983df into AliceO2Group:master Jul 30, 2025
12 of 13 checks passed
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.

4 participants