Skip to content

[PWGLF] Nuclei proton correlation: fix proton rejection - #18220

Merged
maciacco merged 1 commit into
AliceO2Group:masterfrom
ercolessi:master
Oct 8, 2026
Merged

maciacco merged 1 commit into
AliceO2Group:masterfrom
ercolessi:master

Conversation

@ercolessi

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions github-actions Bot added the pwglf label Oct 8, 2026
@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown

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

@ercolessi
ercolessi marked this pull request as ready for review October 8, 2026 12:37
@maciacco
maciacco enabled auto-merge (squash) October 8, 2026 12:39
@maciacco
maciacco merged commit cb91f76 into AliceO2Group:master Oct 8, 2026
14 of 15 checks passed
@alibuild

alibuild commented Oct 8, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 6ec2fbe at 2026-10-08 18:33:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 2 errors and 1 warnings.'
Found 2 errors and 1 warnings.
++ [[ 2 -gt 0 ]]
++ cat /sw/BUILD/feb2ec0b717bf5131a7e292edcda84191cd42a36/O2Physics-code-check/errors.txt
PWGLF/Tasks/Nuspex/hadronnucleicorrelation.cxx:1972:40: error: std::move of the expression of the trivially-copyable type 'value_type' (aka 'GenCoalescenceCandidate') has no effect [performance-move-const-arg,-warnings-as-errors]
PWGLF/Tasks/Nuspex/hadronnucleicorrelation.cxx:1977:38: error: std::move of the variable 'deuteron' of the trivially-copyable type 'GenCoalescenceCandidate' has no effect [performance-move-const-arg,-warnings-as-errors]
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/feb2ec0b717bf5131a7e292edcda84191cd42a36/O2Physics-code-check/warnings.txt
PWGLF/Tasks/Nuspex/hadronnucleicorrelation.cxx:1751:22: warning: use designated initializer list to initialize 'GenCandidate' [modernize-use-designated-initializers]
++ [[ 2 -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

Labels

Development

Successfully merging this pull request may close these issues.

3 participants