Skip to content

[PWGEM] PhotonMeson: Fix bug in photonconversionbuilder `v0photoncand…#16966

Merged
mhemmer-cern merged 3 commits into
AliceO2Group:masterfrom
mhemmer-cern:pwgem-pm-clang-tidy
Jul 8, 2026
Merged

[PWGEM] PhotonMeson: Fix bug in photonconversionbuilder `v0photoncand…#16966
mhemmer-cern merged 3 commits into
AliceO2Group:masterfrom
mhemmer-cern:pwgem-pm-clang-tidy

Conversation

@mhemmer-cern

Copy link
Copy Markdown
Collaborator

…idate.setPhotonCandidate`

  • Fix a bug of swapped arugments in the function call v0photoncandidate.setPhotonCandidate where eledcaXY and posdcaZ were swapped.
  • Fix clang-tidy warnings and errors in photonconversionbuilder

…idate.setPhotonCandidate`

- Fix a bug of swapped arugments in the function call `v0photoncandidate.setPhotonCandidate` where `eledcaXY` and `posdcaZ` were swapped.
- Fix clang-tidy warnings and errors in photonconversionbuilder
@github-actions github-actions Bot added pwgem photon Photon and neutral meson PAG label labels Jul 8, 2026
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

O2 linter results: ❌ 38 errors, ⚠️ 71 warnings, 🔕 1 disabled

@mhemmer-cern mhemmer-cern enabled auto-merge (squash) July 8, 2026 14:59
alibuild
alibuild previously approved these changes Jul 8, 2026

@alibuild alibuild left a comment

Copy link
Copy Markdown
Collaborator

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 @mhemmer-cern.

@alibuild

alibuild commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 62adc94 at 2026-07-08 17:07:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 3 errors and 0 warnings.'
Found 3 errors and 0 warnings.
++ [[ 3 -gt 0 ]]
++ cat /sw/BUILD/233db9596aded1bbcf502eb9e659f484e3d549f8/O2Physics-code-check/errors.txt
PWGEM/PhotonMeson/Core/EmMlResponsePCM.h:38:9: error: function-like macro 'CHECK_AND_FILL_VEC_PCM_FULL' used; consider a 'constexpr' template function [cppcoreguidelines-macro-usage,-warnings-as-errors]
PWGEM/PhotonMeson/Core/EmMlResponsePCM.h:40:32: error: macro argument should be enclosed in parentheses [bugprone-macro-parentheses,-warnings-as-errors]
PWGEM/PhotonMeson/Core/EmMlResponsePCM.h:46:9: error: function-like macro 'CHECK_AND_FILL_VEC_PCM' used; consider a 'constexpr' template function [cppcoreguidelines-macro-usage,-warnings-as-errors]
++ [[ 0 -gt 0 ]]
++ [[ 3 -gt 0 ]]
++ exit 1
--

Full log here.

@mhemmer-cern mhemmer-cern disabled auto-merge July 8, 2026 15:42

@alibuild alibuild left a comment

Copy link
Copy Markdown
Collaborator

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 @mhemmer-cern.

@mhemmer-cern mhemmer-cern merged commit 710ee27 into AliceO2Group:master Jul 8, 2026
14 of 16 checks passed
@mhemmer-cern mhemmer-cern deleted the pwgem-pm-clang-tidy branch July 8, 2026 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

photon Photon and neutral meson PAG label pwgem

Development

Successfully merging this pull request may close these issues.

2 participants