Skip to content

[PWGLF,PWGMM] Update phi selection for data and mc - #18296

Merged
gbencedi merged 3 commits into
AliceO2Group:masterfrom
gbencedi:master
Oct 16, 2026
Merged

gbencedi merged 3 commits into
AliceO2Group:masterfrom
gbencedi:master

Conversation

@gbencedi

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions

Copy link
Copy Markdown

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

@github-actions github-actions Bot changed the title Update phi selection for data and mc [PWGLF,PWGMM] Update phi selection for data and mc Oct 14, 2026
@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 9aeab25 at 2026-10-14 20:12:

No log files found

Full log here.

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for 9aeab25 at 2026-10-14 20:12:

No log files found

Full log here.

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 9aeab25 at 2026-10-14 20:14:

No log files found

Full log here.

@gbencedi
gbencedi enabled auto-merge (squash) October 15, 2026 16:31
alibuild
alibuild previously approved these changes Oct 15, 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 @gbencedi.

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for b981a80 at 2026-10-16 16:42:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'To reproduce the results below locally, run: '\''aliBuild build O2Physics-code-check -e ALIBUILD_BASE_HASH=upstream/master'\''.'
To reproduce the results below locally, run: 'aliBuild build O2Physics-code-check -e ALIBUILD_BASE_HASH=upstream/master'.
++ echo 'To apply all automatic fixes, add the '\''-e O2PHYSICS_CHECKER_FIX=1'\'' option.'
To apply all automatic fixes, add the '-e O2PHYSICS_CHECKER_FIX=1' option.
++ echo 'See the documentation for more details at: <https://aliceo2group.github.io/analysis-framework/docs/tools/#running-with-o2physics-code-check>.'
See the documentation for more details at: <https://aliceo2group.github.io/analysis-framework/docs/tools/#running-with-o2physics-code-check>.
++ echo 'Found 1 errors and 1 warnings.'
Found 1 errors and 1 warnings.
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/c2ea020b0ebb32c1dab3817d092ff10a13a065ef/O2Physics-code-check/errors.txt
PWGMM/Mult/Tasks/dndetaMFTPbPb.cxx:982:21: error: no header providing "o2::math_utils::bringTo02Pi" is directly included [misc-include-cleaner,-warnings-as-errors]
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/c2ea020b0ebb32c1dab3817d092ff10a13a065ef/O2Physics-code-check/warnings.txt
PWGMM/Mult/Tasks/dndetaMFTPbPb.cxx:2694:33: warning: boolean expression can be simplified by DeMorgan's theorem [readability-simplify-boolean-expr]
++ [[ 1 -gt 0 ]]
++ exit 1
--

Full log here.

included MathUtils/Utils.h
@gbencedi
gbencedi enabled auto-merge (squash) October 16, 2026 17:45

@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 @gbencedi.

@gbencedi
gbencedi merged commit 425c812 into AliceO2Group:master Oct 16, 2026
13 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants