Skip to content

[PWGDQ] update AcrossTFMixing for polarization#16932

Open
zjxiongOvO wants to merge 1 commit into
AliceO2Group:masterfrom
zjxiongOvO:master
Open

[PWGDQ] update AcrossTFMixing for polarization#16932
zjxiongOvO wants to merge 1 commit into
AliceO2Group:masterfrom
zjxiongOvO:master

Conversation

@zjxiongOvO

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the pwgdq label Jul 6, 2026
@github-actions github-actions Bot changed the title update AcrossTFMixing for polarization [PWGDQ] update AcrossTFMixing for polarization Jul 6, 2026
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

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

@alibuild

alibuild commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 8e9ebe7 at 2026-07-06 15:03:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 146 errors and 50 warnings.'
Found 146 errors and 50 warnings.
++ [[ 146 -gt 0 ]]
++ cat /sw/BUILD/d827a026e6c897d888d14d667ceede3e9564f2cc/O2Physics-code-check/errors.txt
PWGDQ/Core/VarManager.h:63:1: error: included header TH3.h is not used directly [misc-include-cleaner,-warnings-as-errors]
PWGDQ/Core/VarManager.h:88:10: error: inclusion of deprecated C++ header 'math.h'; consider using 'cmath' instead [modernize-deprecated-headers,-warnings-as-errors]
PWGDQ/Core/VarManager.h:181:3: error: initial values in enum 'VarManager::BarrelTrackFilteringBits' are not consistent, consider explicit initialization of all, none or only the first enumerator [readability-enum-initial-value,-warnings-as-errors]
PWGDQ/Core/VarManager.h:197:2: error: redundant access specifier has the same accessibility as the previous access specifier [readability-redundant-access-specifiers,-warnings-as-errors]
PWGDQ/Core/VarManager.h:198:3: error: initial values in enum 'VarManager::Variables' are not consistent, consider explicit initialization of all, none or only the first enumerator [readability-enum-initial-value,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1190:10: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1191:10: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1211:49: error: the const qualified parameter 'usedVars' is copied for each invocation; consider making it a reference [performance-unnecessary-value-param,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1392:15: error: function 'VarManager::FillTwoEvents' has a definition with different parameter names [readability-inconsistent-declaration-parameter-name,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1394:15: error: function 'VarManager::FillTwoMixEvents' has a definition with different parameter names [readability-inconsistent-declaration-parameter-name,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1398:15: error: function 'VarManager::FillTwoMixEventsCumulants' has a definition with different parameter names [readability-inconsistent-declaration-parameter-name,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1402:15: error: function 'VarManager::FillPhoton' has a definition with different parameter names [readability-inconsistent-declaration-parameter-name,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1440:15: error: function 'VarManager::FillQuadMC' has a definition with different parameter names [readability-inconsistent-declaration-parameter-name,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1496:18: error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1496:71: error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1500:18: error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1500:67: error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1504:18: error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1504:67: error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1508:18: error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1508:69: error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1530:14: error: use nullptr [modernize-use-nullptr,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1531:7: error: do not use 'else' after 'return' [readability-else-after-return,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1553:2: error: redundant access specifier has the same accessibility as the previous access specifier [readability-redundant-access-specifiers,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1557:10: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1561:10: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1605:10: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1653:3: error: uninitialized record type: 'trkpos_par' [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1654:3: error: uninitialized record type: 'trkmom_par' [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1655:3: error: uninitialized record type: 'trk_cov' [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1659:3: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1661:3: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1666:26: 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]
PWGDQ/Core/VarManager.h:1667:32: 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]
PWGDQ/Core/VarManager.h:1679:3: error: uninitialized record type: 'trk_cov' [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1681:3: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1683:3: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1688:26: 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]
PWGDQ/Core/VarManager.h:1689:32: 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]
PWGDQ/Core/VarManager.h:1742:51: error: passing result of std::move() as a const reference argument; no move will actually happen [performance-move-const-arg,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1742:70: error: std::move of the variable 'vtxCov' of the trivially-copyable type 'std::array<float, 6>' has no effect; remove std::move() [performance-move-const-arg,-warnings-as-errors]
PWGDQ/Core/VarManager.h:1798:14: 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]
[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

Labels

Development

Successfully merging this pull request may close these issues.

2 participants