Skip to content

[PWGJE] Add time reclustering, plus centrality selection and event subtractio… - #17329

Open
zovarga wants to merge 2 commits into
AliceO2Group:masterfrom
zovarga:zvarga-lund-plane-centrality
Open

[PWGJE] Add time reclustering, plus centrality selection and event subtractio…#17329
zovarga wants to merge 2 commits into
AliceO2Group:masterfrom
zovarga:zvarga-lund-plane-centrality

Conversation

@zovarga

@zovarga zovarga commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

…n for light/heavy-ion processing

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown

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

@github-actions github-actions Bot changed the title Add time reclustering, plus centrality selection and event subtractio… [PWGJE] Add time reclustering, plus centrality selection and event subtractio… Aug 5, 2026
@alibuild

alibuild commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for 6fd112f at 2026-08-05 08:07:

## sw/BUILD/O2-latest/log
CMake Error in Detectors/Base/CMakeLists.txt:

Full log here.

@alibuild

alibuild commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 6fd112f at 2026-08-05 08:08:

## sw/BUILD/O2-latest/log
CMake Error in Detectors/Base/CMakeLists.txt:

Full log here.

@alibuild

alibuild commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 6fd112f at 2026-08-05 09:51:

## sw/BUILD/O2-latest/log
CMake Error in Detectors/Base/CMakeLists.txt:

Full log here.

@zovarga
zovarga force-pushed the zvarga-lund-plane-centrality branch from 6fd112f to 86dad21 Compare August 5, 2026 14:10
@alibuild

alibuild commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 86dad21 at 2026-08-05 18:04:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 6 errors and 0 warnings.'
Found 6 errors and 0 warnings.
++ [[ 6 -gt 0 ]]
++ cat /sw/BUILD/b44cd41bfc05f2053160914e67b975c3e3a6601f/O2Physics-code-check/errors.txt
PWGJE/Tasks/jetLundPlane.cxx:225:72: error: forwarding reference parameter 'constituents' is never forwarded inside the function body [cppcoreguidelines-missing-std-forward,-warnings-as-errors]
PWGJE/Tasks/jetLundPlane.cxx:639:42: error: use designated initializer list to initialize 'McEventInfo' [modernize-use-designated-initializers,-warnings-as-errors]
PWGJE/Tasks/jetLundPlane.cxx:793:34: error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]
PWGJE/Tasks/jetLundPlane.cxx:888:9: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGJE/Tasks/jetLundPlane.cxx:890:18: error: 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,-warnings-as-errors]
PWGJE/Tasks/jetLundPlane.cxx:907:27: error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]
++ [[ 0 -gt 0 ]]
++ [[ 6 -gt 0 ]]
++ exit 1
--

Full log here.

@nzardosh
nzardosh enabled auto-merge (squash) August 6, 2026 08:09
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