Skip to content

[PWGJE] adding bit for derived data trigger tracks - #17458

Merged
nzardosh merged 1 commit into
AliceO2Group:masterfrom
nzardosh:ttbit
Aug 15, 2026
Merged

[PWGJE] adding bit for derived data trigger tracks#17458
nzardosh merged 1 commit into
AliceO2Group:masterfrom
nzardosh:ttbit

Conversation

@nzardosh

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions github-actions Bot changed the title adding bit for derived data trigger tracks [PWGJE] adding bit for derived data trigger tracks Aug 14, 2026
@github-actions

Copy link
Copy Markdown

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

@nzardosh
nzardosh enabled auto-merge (squash) August 14, 2026 13:36

@alibuild alibuild 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.

Auto-approving on behalf of @nzardosh.

@alibuild

alibuild commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for efbde6a at 2026-08-15 00:47:

## sw/BUILD/O2Physics-latest/log
c++: fatal error: Killed signal terminated program cc1plus
c++: fatal error: Killed signal terminated program cc1plus
ninja: build stopped: subcommand failed.

Full log here.

@alibuild

alibuild commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for efbde6a at 2026-08-15 04:27:

## sw/BUILD/O2Physics-latest/log
c++: fatal error: Killed signal terminated program cc1plus
c++: fatal error: Killed signal terminated program cc1plus
ninja: build stopped: subcommand failed.

Full log here.

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for efbde6a at 2026-08-14 21:06:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 22 errors and 0 warnings.'
Found 22 errors and 0 warnings.
++ [[ 22 -gt 0 ]]
++ cat /sw/BUILD/ce1354f24b8f3ed4a42024a9c58e3fddf972082b/O2Physics-code-check/errors.txt
PWGJE/Core/JetDerivedDataUtilities.h:66:157: error: the parameter 'rctLabel' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]
PWGJE/Core/JetDerivedDataUtilities.h:77:7: error: the 'empty' method should be used to check for emptiness instead of 'size' [readability-container-size-empty,-warnings-as-errors]
PWGJE/Core/JetDerivedDataUtilities.h:99:94: error: the parameter 'selection' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]
PWGJE/Core/JetDerivedDataUtilities.h:308:7: error: the 'empty' method should be used to check for emptiness instead of 'size' [readability-container-size-empty,-warnings-as-errors]
PWGJE/Core/JetDerivedDataUtilities.h:328:78: error: the parameter 'trigger' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]
PWGJE/Core/JetDerivedDataUtilities.h:508:5: error: do not use 'else' after 'return' [readability-else-after-return,-warnings-as-errors]
PWGJE/Core/JetDerivedDataUtilities.h:650:12: error: redundant boolean literal in conditional return statement [readability-simplify-boolean-expr,-warnings-as-errors]
PWGJE/Core/JetDerivedDataUtilities.h:674:5: error: do not use 'else' after 'return' [readability-else-after-return,-warnings-as-errors]
PWGJE/Core/JetDerivedDataUtilities.h:736:65: error: the const qualified parameter 'clusterDefinitions' is copied for each invocation; consider making it a reference [performance-unnecessary-value-param,-warnings-as-errors]
PWGJE/Core/JetDerivedDataUtilities.h:743:10: error: variable 'end' is not initialized [cppcoreguidelines-init-variables,-warnings-as-errors]
PWGJE/DataModel/JetReducedData.h:304:32: error: do not use 'else' after 'return' [readability-else-after-return,-warnings-as-errors]
PWGJE/DataModel/JetReducedData.h:389:180: error: do not use 'else' after 'return' [readability-else-after-return,-warnings-as-errors]
PWGJE/DataModel/JetReducedData.h:391:199: error: do not use 'else' after 'return' [readability-else-after-return,-warnings-as-errors]
PWGJE/DataModel/JetReducedData.h:393:201: error: do not use 'else' after 'return' [readability-else-after-return,-warnings-as-errors]
PWGJE/TableProducer/derivedDataSelector.cxx:158:15: error: loop variable is copied but only used as const reference; consider making it a const reference [performance-for-range-copy,-warnings-as-errors]
PWGJE/TableProducer/derivedDataSelector.cxx:160:17: error: loop variable is copied but only used as const reference; consider making it a const reference [performance-for-range-copy,-warnings-as-errors]
PWGJE/TableProducer/derivedDataWriter.cxx:50:8: error: constructor does not initialize these fields: precisionPositionMask, precisionMomentumMask [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGJE/TableProducer/derivedDataWriter.cxx:487:11: error: no header providing "SETBIT" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGJE/TableProducer/derivedDataWriter.cxx:714:19: error: loop variable is copied but only used as const reference; consider making it a const reference [performance-for-range-copy,-warnings-as-errors]
PWGJE/TableProducer/derivedDataWriter.cxx:718:19: error: loop variable is copied but only used as const reference; consider making it a const reference [performance-for-range-copy,-warnings-as-errors]
PWGJE/TableProducer/derivedDataWriter.cxx:721:11: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGJE/TableProducer/derivedDataWriter.cxx:941:9: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
++ [[ 0 -gt 0 ]]
++ [[ 22 -gt 0 ]]
++ exit 1
--

Full log here.

@nzardosh
nzardosh merged commit f0b0c88 into AliceO2Group:master Aug 15, 2026
12 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.

2 participants