Skip to content

[PWGLF] Fix case for neutron in histogram filling - #17977

Merged
maciacco merged 1 commit into
AliceO2Group:masterfrom
njacazio:master
Sep 20, 2026
Merged

maciacco merged 1 commit into
AliceO2Group:masterfrom
njacazio:master

Conversation

@njacazio

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions github-actions Bot added the pwglf label Sep 20, 2026
@github-actions github-actions Bot changed the title Fix case for neutron in histogram filling [PWGLF] Fix case for neutron in histogram filling Sep 20, 2026
@njacazio njacazio closed this Sep 20, 2026
@github-actions

Copy link
Copy Markdown

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

@njacazio njacazio reopened this Sep 20, 2026
@njacazio
njacazio marked this pull request as ready for review September 20, 2026 06:43
@maciacco
maciacco enabled auto-merge (squash) September 20, 2026 06:58
@maciacco
maciacco merged commit 17f4bba into AliceO2Group:master Sep 20, 2026
20 of 22 checks passed
@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for a4095b2 at 2026-09-20 12:57:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 1 errors and 3 warnings.'
Found 1 errors and 3 warnings.
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/70e8132507003ce2c1eb4732adaed4d913e5a23f/O2Physics-code-check/errors.txt
PWGLF/Tasks/Nuspex/hadronnucleicorrelation.cxx:2011:12: error: no header providing "size_t" is directly included [misc-include-cleaner,-warnings-as-errors]
++ [[ 3 -gt 0 ]]
++ cat /sw/BUILD/70e8132507003ce2c1eb4732adaed4d913e5a23f/O2Physics-code-check/warnings.txt
PWGLF/Tasks/Nuspex/hadronnucleicorrelation.cxx:188:5: warning: function 'pt' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGLF/Tasks/Nuspex/hadronnucleicorrelation.cxx:189:5: warning: function 'eta' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGLF/Tasks/Nuspex/hadronnucleicorrelation.cxx:190:5: warning: function 'phi' should be marked [[nodiscard]] [modernize-use-nodiscard]
++ [[ 1 -gt 0 ]]
++ exit 1
--

Full log here.

@vkucera

vkucera commented Sep 20, 2026

Copy link
Copy Markdown
Collaborator

@njacazio Please fix the error.

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