[PWGJE] Fix physical primary V0 selection in MC and add V0 jet spectra vs multiplicity task by ercolessi · Pull Request #18594 · AliceO2Group/O2Physics

ercolessi · 2026-11-04T17:17:09Z

Hi,

This PR introduces two main changes:

A small fix for V0 clustering in analyseV0s (MC Particle-Level): analyseV0s now skips non-physical-primary V0s on MC particle-level tables.
Previously, Lambda particles originating from Xi+- / Omega+- decays were clustered together with their primary charged mother, leading to double-counting of their momentum in the jet pT.
Detector-level and data remain unchanged.
Note: This changes the default behavior for all users of the MCP V0 jet finder.
New task lf-jet-spectra to measures K0s, Lambda, and Lambda-bar pT spectra in jets and in the UE vs FT0M multiplicity.
It follows the same principle as PWGLF strangenessInJets, but integrates JET tools to include the V0 pT directly in the jet clustering as suggested by PWGJE convenors.

Cheers,
Francesca

fix fix

github-actions · 2026-11-04T17:17:29Z

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

alibuild · 2026-11-04T17:37:44Z

Error while checking build/O2Physics/o2 for 34fedc2 at 2026-11-04 18:37:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGJE/Tasks/lfJetSpectra.cxx:376:64: error: arithmetic between enumeration type 'LfJetSpectra::V0Origin' and floating-point type 'double' is deprecated [-Werror=deprecated-enum-float-conversion]
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGJE/Tasks/lfJetSpectra.cxx:403:64: error: arithmetic between enumeration type 'LfJetSpectra::V0Source' and floating-point type 'double' is deprecated [-Werror=deprecated-enum-float-conversion]
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGJE/Tasks/lfJetSpectra.cxx:469:129: error: arithmetic between enumeration type 'LfJetSpectra::V0SelectionStep' and floating-point type 'double' is deprecated [-Werror=deprecated-enum-float-conversion]
ninja: build stopped: subcommand failed.

Full log here.

alibuild · 2026-11-04T17:41:49Z

Error while checking build/O2Physics/staging for 34fedc2 at 2026-11-04 18:41:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGJE/Tasks/lfJetSpectra.cxx:376:64: error: arithmetic between enumeration type 'LfJetSpectra::V0Origin' and floating-point type 'double' is deprecated [-Werror=deprecated-enum-float-conversion]
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGJE/Tasks/lfJetSpectra.cxx:403:64: error: arithmetic between enumeration type 'LfJetSpectra::V0Source' and floating-point type 'double' is deprecated [-Werror=deprecated-enum-float-conversion]
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGJE/Tasks/lfJetSpectra.cxx:469:129: error: arithmetic between enumeration type 'LfJetSpectra::V0SelectionStep' and floating-point type 'double' is deprecated [-Werror=deprecated-enum-float-conversion]
ninja: build stopped: subcommand failed.

Full log here.

alibuild · 2026-11-04T18:05:35Z

Error while checking build/O2Physics/code-check for 34fedc2 at 2026-11-04 19:05:

## 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 12 errors and 24 warnings.'
Found 12 errors and 24 warnings.
++ [[ 12 -gt 0 ]]
++ cat /sw/BUILD/76cd8333ec114a947634f6148c547d9c4d31a551/O2Physics-code-check/errors.txt
PWGJE/Core/JetFindingUtilities.h:41:1: error: included header utility is not used directly [misc-include-cleaner,-warnings-as-errors]
PWGJE/Tasks/lfJetSpectra.cxx:25:1: error: included header LFStrangenessTables.h is not used directly [misc-include-cleaner,-warnings-as-errors]
PWGJE/Tasks/lfJetSpectra.cxx:26:1: error: included header V0SelectorTables.h is not used directly [misc-include-cleaner,-warnings-as-errors]
PWGJE/Tasks/lfJetSpectra.cxx:77:45: error: no header providing "o2::aod::McCollisions" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGJE/Tasks/lfJetSpectra.cxx:80:54: error: no header providing "o2::framework::OutputObjHandlingPolicy" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGJE/Tasks/lfJetSpectra.cxx:196:3: error: no header providing "o2::framework::Service" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGJE/Tasks/lfJetSpectra.cxx:234:17: error: no header providing "o2::aod::McParticles" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGJE/Tasks/lfJetSpectra.cxx:234:64: error: no header providing "o2::aod::mcparticle::mcCollisionId" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGJE/Tasks/lfJetSpectra.cxx:236:73: error: no header providing "o2::aod::jv0indices::mcCollisionId" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGJE/Tasks/lfJetSpectra.cxx:376:64: error: arithmetic between enumeration type 'LfJetSpectra::V0Origin' and floating-point type 'double' is deprecated [clang-diagnostic-deprecated-enum-float-conversion,-warnings-as-errors]
PWGJE/Tasks/lfJetSpectra.cxx:403:64: error: arithmetic between enumeration type 'LfJetSpectra::V0Source' and floating-point type 'double' is deprecated [clang-diagnostic-deprecated-enum-float-conversion,-warnings-as-errors]
PWGJE/Tasks/lfJetSpectra.cxx:469:129: error: arithmetic between enumeration type 'LfJetSpectra::V0SelectionStep' and floating-point type 'double' is deprecated [clang-diagnostic-deprecated-enum-float-conversion,-warnings-as-errors]
++ [[ 24 -gt 0 ]]
++ cat /sw/BUILD/76cd8333ec114a947634f6148c547d9c4d31a551/O2Physics-code-check/warnings.txt
PWGJE/Core/JetFindingUtilities.h:44:10: warning: inclusion of deprecated C++ header 'math.h'; consider using 'cmath' instead [modernize-deprecated-headers]
PWGJE/Core/JetFindingUtilities.h:272:12: warning: redundant boolean literal in conditional return statement [readability-simplify-boolean-expr]
PWGJE/Core/JetFindingUtilities.h:273:5: warning: do not use 'else' after 'return' [readability-else-after-return]
PWGJE/Core/JetFindingUtilities.h:353:82: warning: repeated branch body in conditional chain [bugprone-branch-clone]
PWGJE/Core/JetFindingUtilities.h:355:7: warning: do not use 'else' after 'continue' [readability-else-after-return]
PWGJE/Core/JetFindingUtilities.h:414:5: warning: do not use 'else' after 'return' [readability-else-after-return]
PWGJE/Tasks/lfJetSpectra.cxx:79:8: warning: constructor does not initialize these fields: pdgDatabase [cppcoreguidelines-pro-type-member-init]
PWGJE/Tasks/lfJetSpectra.cxx:464:13: warning: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays]
PWGJE/Tasks/lfJetSpectra.cxx:844:35: warning: use 'contains' to check for membership [readability-container-contains]
PWGJE/Tasks/lfJetSpectra.cxx:879:14: warning: implicit conversion 'size_type' (aka 'unsigned long') -> 'bool' [readability-implicit-bool-conversion]
PWGJE/Tasks/lfJetSpectra.cxx:879:27: warning: use 'contains' to check for membership [readability-container-contains]
PWGJE/Tasks/lfJetSpectra.cxx:879:54: warning: implicit conversion 'size_type' (aka 'unsigned long') -> 'bool' [readability-implicit-bool-conversion]
PWGJE/Tasks/lfJetSpectra.cxx:879:64: warning: use 'contains' to check for membership [readability-container-contains]
PWGJE/Tasks/lfJetSpectra.cxx:1100:35: warning: use 'contains' to check for membership [readability-container-contains]
PWGJE/Tasks/lfJetSpectra.cxx:1141:14: warning: implicit conversion 'size_type' (aka 'unsigned long') -> 'bool' [readability-implicit-bool-conversion]
PWGJE/Tasks/lfJetSpectra.cxx:1141:27: warning: use 'contains' to check for membership [readability-container-contains]
PWGJE/Tasks/lfJetSpectra.cxx:1141:54: warning: implicit conversion 'size_type' (aka 'unsigned long') -> 'bool' [readability-implicit-bool-conversion]
PWGJE/Tasks/lfJetSpectra.cxx:1141:64: warning: use 'contains' to check for membership [readability-container-contains]
PWGJE/Tasks/lfJetSpectra.cxx:1220:14: warning: implicit conversion 'size_type' (aka 'unsigned long') -> 'bool' [readability-implicit-bool-conversion]
PWGJE/Tasks/lfJetSpectra.cxx:1220:27: warning: use 'contains' to check for membership [readability-container-contains]
PWGJE/Tasks/lfJetSpectra.cxx:1220:54: warning: implicit conversion 'size_type' (aka 'unsigned long') -> 'bool' [readability-implicit-bool-conversion]
PWGJE/Tasks/lfJetSpectra.cxx:1220:64: warning: use 'contains' to check for membership [readability-container-contains]
[0 more errors; see full log]

Full log here.

alibuild · 2026-11-05T09:01:44Z

Error while checking build/O2Physics/o2 for 7f798ee at 2026-11-05 10:01:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGJE/Tasks/lfJetSpectra.cxx:346:45: error: comparison of floating-point type 'float' with enumeration type 'PDG_t' is deprecated [-Werror=deprecated-enum-float-conversion]
ninja: build stopped: subcommand failed.

Full log here.

alibuild · 2026-11-05T09:13:59Z

Error while checking build/O2Physics/staging for 7f798ee at 2026-11-05 10:13:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGJE/Tasks/lfJetSpectra.cxx:346:45: error: comparison of floating-point type 'float' with enumeration type 'PDG_t' is deprecated [-Werror=deprecated-enum-float-conversion]
ninja: build stopped: subcommand failed.

Full log here.

alibuild · 2026-11-05T09:30:04Z

Error while checking build/O2Physics/code-check for 7f798ee at 2026-11-05 10:30:

## 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 2 errors and 7 warnings.'
Found 2 errors and 7 warnings.
++ [[ 2 -gt 0 ]]
++ cat /sw/BUILD/6b67c115bde7fa42ad67e905f68ce4a37379c837/O2Physics-code-check/errors.txt
PWGJE/Core/JetFindingUtilities.h:41:1: error: included header utility is not used directly [misc-include-cleaner,-warnings-as-errors]
PWGJE/Tasks/lfJetSpectra.cxx:346:45: error: comparison of floating-point type 'float' with enumeration type 'PDG_t' is deprecated [clang-diagnostic-deprecated-enum-float-conversion,-warnings-as-errors]
++ [[ 7 -gt 0 ]]
++ cat /sw/BUILD/6b67c115bde7fa42ad67e905f68ce4a37379c837/O2Physics-code-check/warnings.txt
PWGJE/Core/JetFindingUtilities.h:44:10: warning: inclusion of deprecated C++ header 'math.h'; consider using 'cmath' instead [modernize-deprecated-headers]
PWGJE/Core/JetFindingUtilities.h:272:12: warning: redundant boolean literal in conditional return statement [readability-simplify-boolean-expr]
PWGJE/Core/JetFindingUtilities.h:273:5: warning: do not use 'else' after 'return' [readability-else-after-return]
PWGJE/Core/JetFindingUtilities.h:353:82: warning: repeated branch body in conditional chain [bugprone-branch-clone]
PWGJE/Core/JetFindingUtilities.h:355:7: warning: do not use 'else' after 'continue' [readability-else-after-return]
PWGJE/Core/JetFindingUtilities.h:414:5: warning: do not use 'else' after 'return' [readability-else-after-return]
PWGJE/Tasks/lfJetSpectra.cxx:82:8: warning: constructor does not initialize these fields: pdgDatabase [cppcoreguidelines-pro-type-member-init]
++ [[ 2 -gt 0 ]]
++ exit 1
--

Full log here.

alibuild · 2026-11-05T10:14:13Z

Error while checking build/O2Physics/code-check for 20a3147 at 2026-11-05 11:14:

## 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 7 warnings.'
Found 1 errors and 7 warnings.
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/caa48915f3faa318d4fa655f326a3b3ae9b8abae/O2Physics-code-check/errors.txt
PWGJE/Core/JetFindingUtilities.h:41:1: error: included header utility is not used directly [misc-include-cleaner,-warnings-as-errors]
++ [[ 7 -gt 0 ]]
++ cat /sw/BUILD/caa48915f3faa318d4fa655f326a3b3ae9b8abae/O2Physics-code-check/warnings.txt
PWGJE/Core/JetFindingUtilities.h:44:10: warning: inclusion of deprecated C++ header 'math.h'; consider using 'cmath' instead [modernize-deprecated-headers]
PWGJE/Core/JetFindingUtilities.h:272:12: warning: redundant boolean literal in conditional return statement [readability-simplify-boolean-expr]
PWGJE/Core/JetFindingUtilities.h:273:5: warning: do not use 'else' after 'return' [readability-else-after-return]
PWGJE/Core/JetFindingUtilities.h:353:82: warning: repeated branch body in conditional chain [bugprone-branch-clone]
PWGJE/Core/JetFindingUtilities.h:355:7: warning: do not use 'else' after 'continue' [readability-else-after-return]
PWGJE/Core/JetFindingUtilities.h:414:5: warning: do not use 'else' after 'return' [readability-else-after-return]
PWGJE/Tasks/lfJetSpectra.cxx:82:8: warning: constructor does not initialize these fields: pdgDatabase [cppcoreguidelines-pro-type-member-init]
++ [[ 1 -gt 0 ]]
++ exit 1
--

Full log here.

nzardosh · 2026-11-06T13:50:35Z

+    registryMC.fill(HIST("number_of_events_vsmultiplicity_rec"), multiplicity, weight);
+
+    // Generated particles of the MC collision associated to the reconstructed collision
+    const auto genV0s = jetParticles.sliceBy(particlesPerMcCollision, mcCollision.mcCollisionId());


I think this should be mcCollision.globalIndex() the other one points to the aod::McCollisions table

Yes thanks!

nzardosh · 2026-11-06T13:50:41Z

+    const float multiplicity = jcoll.centFT0M();
+    registryData.fill(HIST("number_of_events_vsmultiplicity"), multiplicity);
+
+    const auto jetFinderV0s = buildDetectorLevelInput(v0s, tracks);


you can get the V0s directly from the jet with
for (auto v0Cand : v0jets.candidate_asaod::CandidatesV0Data())

If I understand correctly your comment here, for the in-jet V0 observables I think the code should already iterate over the jet constituents directly using CandidatesV0Data. The loop over the full v0s table here is intentional to fill the inclusive QA once per candidate, including candidates that are not jet constituents. The UE loop also, bc it needs to consider V0s outside the jet.

nzardosh · 2026-11-06T13:50:48Z

+      return false;
+    }
+    for (const auto& v0 : v0s) {
+      if (jetcandidateutilities::isDaughterTrack(track, v0)) {


if you take the jet constituents this is already removed by the jet finder. But if you take the full JTracks it is not

Yes this is always used with JetTracks

nzardosh · 2026-11-06T13:50:59Z

+using MatchedMCPV0JetsWithConstituents = soa::Join<aod::V0ChargedMCParticleLevelJets, aod::V0ChargedMCParticleLevelJetConstituents, aod::V0ChargedMCParticleLevelJetsMatchedToV0ChargedMCDetectorLevelJets>;
+
+// PWGJE MC collisions and their indices to the original AO2D MC collisions.
+using JetMcCollisionsWithPIs = soa::Join<aod::JetMcCollisions, aod::JMcCollisionPIs>;


aod::JMcCollisionPIs
this you could remove since you don't link back to the original table anymore

nzardosh · 2026-11-06T13:54:11Z

+
+    // Reconstructed collisions of this MC collision passing the event selection
+    std::unordered_set<int> selectedCollisions;
+    for (const auto& collision : collisions) {


does smallGroups already group these with the JetMcCollisions?

Yes I think SmallGroups here gives the reconstructed collisions associated with the current JetMcCollision

nzardosh · 2026-11-06T13:54:26Z

+      }
+    }
+
+    const auto jetFinderV0s = buildParticleLevelInput(v0s, particles);


same here with getting them directly from the jet

Same here I think for the in-jet generated V0s the loop uses jet.candidates_asaod::CandidatesV0MCP() to retrieve the V0 constituents. The loop over v0s is for the UE, since those are not constituents of the current jet it filters them to those accepted by the jet finder and excludes V0s belonging to jets above threshold. This was the rationale

alibuild · 2026-11-06T14:34:46Z

Error while checking build/O2Physics/code-check for 3e2d0a8 at 2026-11-06 15:34:

## 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 2 errors and 7 warnings.'
Found 2 errors and 7 warnings.
++ [[ 2 -gt 0 ]]
++ cat /sw/BUILD/7c7c611322b374adb19dee9172863fad8bc38c90/O2Physics-code-check/errors.txt
PWGJE/Core/JetFindingUtilities.h:41:1: error: included header utility is not used directly [misc-include-cleaner,-warnings-as-errors]
PWGJE/Tasks/lfJetSpectra.cxx:26:1: error: included header mcCentrality.h is not used directly [misc-include-cleaner,-warnings-as-errors]
++ [[ 7 -gt 0 ]]
++ cat /sw/BUILD/7c7c611322b374adb19dee9172863fad8bc38c90/O2Physics-code-check/warnings.txt
PWGJE/Core/JetFindingUtilities.h:44:10: warning: inclusion of deprecated C++ header 'math.h'; consider using 'cmath' instead [modernize-deprecated-headers]
PWGJE/Core/JetFindingUtilities.h:272:12: warning: redundant boolean literal in conditional return statement [readability-simplify-boolean-expr]
PWGJE/Core/JetFindingUtilities.h:273:5: warning: do not use 'else' after 'return' [readability-else-after-return]
PWGJE/Core/JetFindingUtilities.h:353:82: warning: repeated branch body in conditional chain [bugprone-branch-clone]
PWGJE/Core/JetFindingUtilities.h:355:7: warning: do not use 'else' after 'continue' [readability-else-after-return]
PWGJE/Core/JetFindingUtilities.h:414:5: warning: do not use 'else' after 'return' [readability-else-after-return]
PWGJE/Tasks/lfJetSpectra.cxx:80:8: warning: constructor does not initialize these fields: pdgDatabase [cppcoreguidelines-pro-type-member-init]
++ [[ 2 -gt 0 ]]
++ exit 1
--

Full log here.

alibuild · 2026-11-06T16:51:48Z

Error while checking build/O2Physics/code-check for 36d531a at 2026-11-06 17:51:

## 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 2 errors and 7 warnings.'
Found 2 errors and 7 warnings.
++ [[ 2 -gt 0 ]]
++ cat /sw/BUILD/a8b30891d88c5425a05e16ea0905f08f4685ebc9/O2Physics-code-check/errors.txt
PWGJE/Core/JetFindingUtilities.h:41:1: error: included header utility is not used directly [misc-include-cleaner,-warnings-as-errors]
PWGJE/Tasks/lfJetSpectra.cxx:26:1: error: included header mcCentrality.h is not used directly [misc-include-cleaner,-warnings-as-errors]
++ [[ 7 -gt 0 ]]
++ cat /sw/BUILD/a8b30891d88c5425a05e16ea0905f08f4685ebc9/O2Physics-code-check/warnings.txt
PWGJE/Core/JetFindingUtilities.h:44:10: warning: inclusion of deprecated C++ header 'math.h'; consider using 'cmath' instead [modernize-deprecated-headers]
PWGJE/Core/JetFindingUtilities.h:272:12: warning: redundant boolean literal in conditional return statement [readability-simplify-boolean-expr]
PWGJE/Core/JetFindingUtilities.h:273:5: warning: do not use 'else' after 'return' [readability-else-after-return]
PWGJE/Core/JetFindingUtilities.h:353:82: warning: repeated branch body in conditional chain [bugprone-branch-clone]
PWGJE/Core/JetFindingUtilities.h:355:7: warning: do not use 'else' after 'continue' [readability-else-after-return]
PWGJE/Core/JetFindingUtilities.h:414:5: warning: do not use 'else' after 'return' [readability-else-after-return]
PWGJE/Tasks/lfJetSpectra.cxx:77:8: warning: constructor does not initialize these fields: pdgDatabase [cppcoreguidelines-pro-type-member-init]
++ [[ 2 -gt 0 ]]
++ exit 1
--

Full log here.

ercolessi added 2 commits November 4, 2026 17:51

Add possibility to remove V0 primaries from jet finder

efb0a1d

fix fix

New task for v0 included in jet vs multiplicity

34fedc2

github-actions Bot added the pwgje label Nov 4, 2026

ercolessi marked this pull request as ready for review November 4, 2026 17:26

ercolessi requested review from alibuild, fjonasALICE, jaimenorman, mhemmer-cern and nzardosh as code owners November 4, 2026 17:26

vkucera marked this pull request as draft November 4, 2026 18:20

fix build error and warnings

7f798ee

ercolessi marked this pull request as ready for review November 5, 2026 08:49

fix build errors

20a3147

use Jet Tables for MC collisions and particles

3e2d0a8

nzardosh reviewed Nov 6, 2026

View reviewed changes

remove JMcCollisionPIs and fix collision index

36d531a

nzardosh self-requested a review November 6, 2026 16:32

nzardosh approved these changes Nov 6, 2026

View reviewed changes

nzardosh enabled auto-merge (squash) November 6, 2026 16:33

nzardosh merged commit 42b110b into AliceO2Group:master Nov 6, 2026
14 of 16 checks passed

Conversation

ercolessi commented Nov 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented Nov 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

alibuild commented Nov 4, 2026

Uh oh!

alibuild commented Nov 4, 2026

Uh oh!

alibuild commented Nov 4, 2026

Uh oh!

alibuild commented Nov 5, 2026

Uh oh!

alibuild commented Nov 5, 2026

Uh oh!

alibuild commented Nov 5, 2026

Uh oh!

alibuild commented Nov 5, 2026

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

alibuild commented Nov 6, 2026

Uh oh!

Uh oh!

alibuild commented Nov 6, 2026

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Milestone

Development

Uh oh!

3 participants

ercolessi commented Nov 4, 2026 •

edited

Loading

github-actions Bot commented Nov 4, 2026 •

edited

Loading