Skip to content

[PWGLF,PWGMM] fix flattenicity: filling histograms error - #17564

Open
PaolaVT wants to merge 28 commits into
AliceO2Group:masterfrom
PaolaVT:master
Open

[PWGLF,PWGMM] fix flattenicity: filling histograms error#17564
PaolaVT wants to merge 28 commits into
AliceO2Group:masterfrom
PaolaVT:master

Conversation

@PaolaVT

@PaolaVT PaolaVT commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Aug 22, 2026

Copy link
Copy Markdown

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

@github-actions github-actions Bot changed the title fix flattenicity: filling histograms error [PWGLF,PWGMM] fix flattenicity: filling histograms error Aug 22, 2026
@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for e7a3043 at 2026-08-22 21:37:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 3 errors and 2 warnings.'
Found 3 errors and 2 warnings.
++ [[ 3 -gt 0 ]]
++ cat /sw/BUILD/4a9f6a5047e316e8d55ff2c673b79356d09f6c2c/O2Physics-code-check/errors.txt
PWGMM/UE/Tasks/flattenicityTask.cxx:18:1: error: included header Multiplicity.h is not used directly [misc-include-cleaner,-warnings-as-errors]
PWGMM/UE/Tasks/flattenicityTask.cxx:510:17: error: no header providing "std::size_t" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGMM/UE/Tasks/flattenicityTask.cxx:511:9: error: no header providing "uint8_t" is directly included [misc-include-cleaner,-warnings-as-errors]
++ [[ 2 -gt 0 ]]
++ cat /sw/BUILD/4a9f6a5047e316e8d55ff2c673b79356d09f6c2c/O2Physics-code-check/warnings.txt
PWGMM/UE/Tasks/flattenicityTask.cxx:509:9: warning: the 'empty' method should be used to check for emptiness instead of 'size' [readability-container-size-empty]
PWGMM/UE/Tasks/flattenicityTask.cxx:519:9: warning: the 'empty' method should be used to check for emptiness instead of 'size' [readability-container-size-empty]
++ [[ 3 -gt 0 ]]
++ exit 1
--

Full log here.

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