Skip to content

[PWGLF] Fix quadratic PID cut and improve Gen process performance - #17935

Merged
ercolessi merged 2 commits into
AliceO2Group:masterfrom
ercolessi:master
Sep 16, 2026
Merged

ercolessi merged 2 commits into
AliceO2Group:masterfrom
ercolessi:master

Conversation

@ercolessi

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions github-actions Bot added the pwglf label Sep 16, 2026
@github-actions

Copy link
Copy Markdown

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

@ercolessi
ercolessi marked this pull request as ready for review September 16, 2026 12:45
@ercolessi
ercolessi enabled auto-merge (squash) September 16, 2026 14:14
@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for dfcb806 at 2026-09-16 20:37:

## 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/71475e044948c3d3c9a86db6994324437f2ed98f/O2Physics-code-check/errors.txt
PWGLF/Tasks/Nuspex/hadronnucleicorrelation.cxx:2008:12: error: no header providing "size_t" is directly included [misc-include-cleaner,-warnings-as-errors]
++ [[ 3 -gt 0 ]]
++ cat /sw/BUILD/71475e044948c3d3c9a86db6994324437f2ed98f/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.

@ercolessi
ercolessi merged commit 2b96fe4 into AliceO2Group:master Sep 16, 2026
13 of 15 checks passed
@vkucera

vkucera commented Sep 16, 2026

Copy link
Copy Markdown
Collaborator

Please fix the code-check 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