Skip to content

[PWGLF] store signed pt of the candidate, swap hardcoded pdg codes for PDG_t - #18235

Merged
maciacco merged 1 commit into
AliceO2Group:masterfrom
GiorgioAlbertoLucia:omegaomega
Oct 9, 2026
Merged

maciacco merged 1 commit into
AliceO2Group:masterfrom
GiorgioAlbertoLucia:omegaomega

Conversation

@GiorgioAlbertoLucia

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Oct 9, 2026

Copy link
Copy Markdown

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

@maciacco
maciacco enabled auto-merge (squash) October 9, 2026 14:22
@alibuild

alibuild commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 03ac1f4 at 2026-10-09 23:01:

## 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 0 warnings.'
Found 1 errors and 0 warnings.
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/8799e820c373ff717fa485872c14ac730cca6d73/O2Physics-code-check/errors.txt
PWGLF/TableProducer/Strangeness/doubleOmegaTreeCreator.cxx:687:43: error: no header providing "PDG_t" is directly included [misc-include-cleaner,-warnings-as-errors]
++ [[ 0 -gt 0 ]]
++ [[ 1 -gt 0 ]]
++ exit 1
--

Full log here.

@maciacco
maciacco merged commit 3599af1 into AliceO2Group:master Oct 9, 2026
14 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.

4 participants