Skip to content

[PWGCF,PWGLF] Add a dedicated pi-hypertriton workflow to LF - #17947

Open
zhengqi-w wants to merge 2 commits into
AliceO2Group:masterfrom
zhengqi-w:codex/lf-pi-hypertriton-workflow
Open

zhengqi-w wants to merge 2 commits into
AliceO2Group:masterfrom
zhengqi-w:codex/lf-pi-hypertriton-workflow

Conversation

@zhengqi-w

Copy link
Copy Markdown
Contributor

Changes:

  1. Add a dedicated π–hypertriton femtoscopy workflow and Data/MC output tables under PWGLF, with variables for offline BDT analysis.
  2. Add processMCWithCollID and MCHypCandsWColl, sharing the MC reconstruction logic with processMC. Keep collision and daughter-track IDs only in the collision-aware MC table.
  3. Update TrackedHypertritonRecoTask (PWGLF) and HadNucleiFemto (PWGCF) to maintain compatibility with the revised MC candidate tables and collision-ID handling.

@github-actions github-actions Bot changed the title Add a dedicated pi-hypertriton workflow to LF [PWGCF,PWGLF] Add a dedicated pi-hypertriton workflow to LF Sep 17, 2026
@github-actions

Copy link
Copy Markdown

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

@mpuccio
mpuccio enabled auto-merge (squash) September 17, 2026 15:35
@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 51ee8df at 2026-09-17 20:26:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 6 errors and 72 warnings.'
Found 6 errors and 72 warnings.
++ [[ 6 -gt 0 ]]
++ cat /sw/BUILD/905435bf71c09467826adcdc57ff9ab19e9b134a/O2Physics-code-check/errors.txt
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:61:1: error: included header memory is not used directly [misc-include-cleaner,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/piHypertritonFemto.cxx:95:3: error: no header providing "o2::framework::Produces" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/piHypertritonFemto.cxx:319:3: error: no header providing "o2::framework::Service" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/piHypertritonFemto.cxx:321:3: error: no header providing "o2::framework::OutputObj" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/piHypertritonFemto.cxx:405:10: error: no header providing "size_t" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/piHypertritonFemto.cxx:420:83: error: no header providing "o2::track::DirOutward" is directly included [misc-include-cleaner,-warnings-as-errors]
++ [[ 72 -gt 0 ]]
++ cat /sw/BUILD/905435bf71c09467826adcdc57ff9ab19e9b134a/O2Physics-code-check/warnings.txt
PWGCF/Femto/FemtoNuclei/TableProducer/HadNucleiFemto.cxx:509:31: warning: initializer for member 'info' is redundant [readability-redundant-member-init]
PWGCF/Femto/FemtoNuclei/TableProducer/HadNucleiFemto.cxx:537:5: warning: function 'pt' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGCF/Femto/FemtoNuclei/TableProducer/HadNucleiFemto.cxx:538:5: warning: function 'eta' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGCF/Femto/FemtoNuclei/TableProducer/HadNucleiFemto.cxx:543:5: warning: function 'phi' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGCF/Femto/FemtoNuclei/TableProducer/HadNucleiFemto.cxx:544:5: warning: function 'sign' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGCF/Femto/FemtoNuclei/TableProducer/HadNucleiFemto.cxx:548:27: warning: initializer for member 'info' is redundant [readability-redundant-member-init]
PWGCF/Femto/FemtoNuclei/TableProducer/HadNucleiFemto.cxx:559:5: warning: function 'pt' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGCF/Femto/FemtoNuclei/TableProducer/HadNucleiFemto.cxx:560:5: warning: function 'eta' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGCF/Femto/FemtoNuclei/TableProducer/HadNucleiFemto.cxx:561:5: warning: function 'phi' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGCF/Femto/FemtoNuclei/TableProducer/HadNucleiFemto.cxx:562:5: warning: function 'sign' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGCF/Femto/FemtoNuclei/TableProducer/HadNucleiFemto.cxx:566:27: warning: initializer for member 'info' is redundant [readability-redundant-member-init]
PWGCF/Femto/FemtoNuclei/TableProducer/HadNucleiFemto.cxx:1932:7: warning: switch has 2 consecutive identical branches [bugprone-branch-clone]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:82:11: warning: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:86:39: warning: 'betheBlochParNames' is a static definition in anonymous namespace; static is redundant here [readability-static-definition-in-anonymous-namespace]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:87:39: warning: 'particleName' is a static definition in anonymous namespace; static is redundant here [readability-static-definition-in-anonymous-namespace]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:92:3: warning: function 'recoPtHe3' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:93:3: warning: function 'recoPhiHe3' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:94:3: warning: function 'recoEtaHe3' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:95:3: warning: function 'recoPtPi' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:96:3: warning: function 'recoPhiPi' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:97:3: warning: function 'recoEtaPi' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:98:3: warning: function 'genPt' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:99:3: warning: function 'genPtHe3' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:100:3: warning: function 'genPhi' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:101:3: warning: function 'genEta' should be marked [[nodiscard]] [modernize-use-nodiscard]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:148:8: warning: constructor does not initialize these fields: ccdb [cppcoreguidelines-pro-type-member-init]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:196:82: warning: do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead [cppcoreguidelines-pro-bounds-array-to-pointer-decay]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:370:5: warning: use auto when initializing with a template cast to avoid duplicating the type name [modernize-use-auto]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:371:43: warning: use nullptr [modernize-use-nullptr]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:485:123: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:539:46: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:582:23: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:587:136: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/TableProducer/Nuspex/hyperRecoTask.cxx:589:138: warning: statement should be inside braces [readability-braces-around-statements]
[0 more errors; see full log]

Full log here.

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