Skip to content

[PWGLF] Move double-cascade inspector to double-omega search analysis - #17536

Merged
romainschotter merged 3 commits into
AliceO2Group:masterfrom
fmazzasc:master
Aug 31, 2026
Merged

[PWGLF] Move double-cascade inspector to double-omega search analysis#17536
romainschotter merged 3 commits into
AliceO2Group:masterfrom
fmazzasc:master

Conversation

@fmazzasc

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions github-actions Bot changed the title Move double-cascade inspector to double-omega search analysis [PWGLF] Move double-cascade inspector to double-omega search analysis Aug 21, 2026
@github-actions

github-actions Bot commented Aug 21, 2026

Copy link
Copy Markdown

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

@alibuild

alibuild commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 99d656e at 2026-08-30 09:25:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 5 errors and 1 warnings.'
Found 5 errors and 1 warnings.
++ [[ 5 -gt 0 ]]
++ cat /sw/BUILD/1bc7e70becf82cf72d42ef36a3688d742e864068/O2Physics-code-check/errors.txt
PWGLF/DataModel/LFDoubleOmegaTables.h:17:1: error: included header cstdint is not used directly [misc-include-cleaner,-warnings-as-errors]
PWGLF/TableProducer/Strangeness/doubleOmegaTreeCreator.cxx:59:84: error: no header providing "o2::aod::TracksDCA" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/TableProducer/Strangeness/doubleOmegaTreeCreator.cxx:127:8: error: constructor does not initialize these fields: ccdb [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGLF/TableProducer/Strangeness/doubleOmegaTreeCreator.cxx:550:27: error: no header providing "o2::math_utils::Point3D" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/TableProducer/Strangeness/doubleOmegaTreeCreator.cxx:1095:12: error: no header providing "size_t" is directly included [misc-include-cleaner,-warnings-as-errors]
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/1bc7e70becf82cf72d42ef36a3688d742e864068/O2Physics-code-check/warnings.txt
PWGLF/TableProducer/Strangeness/doubleOmegaTreeCreator.cxx:355:29: warning: use designated initializer list to initialize 'LambdaCandidate' [modernize-use-designated-initializers]
++ [[ 5 -gt 0 ]]
++ exit 1
--

Full log here.

@romainschotter romainschotter left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @fmazzasc !
I am not entirely sure to understand why it is necessary to re-build the Lambda and Omega candidates. Why not using the V0 and cascade tables produced by the strangeness builder?
Could you please also add:

  • #include in doubleOmegaTreeCreator.cxx?
    Thank you very much!

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for 99d656e at 2026-08-27 02:04:

No log files found

Full log here.

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 99d656e at 2026-08-30 03:27:

No log files found

Full log here.

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 657863b at 2026-08-31 10:14:

No log files found

Full log here.

@romainschotter

Copy link
Copy Markdown
Collaborator

As discussed privately with @fmazzasc , I understand now that the Lambda and Omega had to be rebuilt because, somehow, the double-Omega signal events were lost in the strangeness building of the propagationService.

However, we should definitely look into it because there are no preselections in the strangeness builder in the propagation service, that should suppress double Omegas. In order to dive deeper into this, it was agreed with @fmazzasc that a switch should be added later this week to use the V0s/cascades from the strangeness builder in the propagation service.

In the meantime, I merge the PR.

Thank you very much!

@romainschotter
romainschotter enabled auto-merge (squash) August 31, 2026 08:18
@romainschotter
romainschotter merged commit b22ad64 into AliceO2Group:master Aug 31, 2026
13 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.

3 participants