Skip to content

[PWGLF] To add the THnSparse for MC closure - #17217

Merged
dmallick2 merged 3 commits into
AliceO2Group:masterfrom
navneetkumar231295:master
Jul 27, 2026
Merged

[PWGLF] To add the THnSparse for MC closure#17217
dmallick2 merged 3 commits into
AliceO2Group:masterfrom
navneetkumar231295:master

Conversation

@navneetkumar231295

Copy link
Copy Markdown
Contributor

Dear Experts in this code, I have added the THnSparse for the MC closure test.

@github-actions

Copy link
Copy Markdown

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

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 14a022c at 2026-07-27 14:06:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 10 errors and 88 warnings.'
Found 10 errors and 88 warnings.
++ [[ 10 -gt 0 ]]
++ cat /sw/BUILD/11a4e675032c0414b4c52fc6d6c16957d7c44faa/O2Physics-code-check/errors.txt
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:77:8: error: constructor does not initialize these fields: ccdb, pdg [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:724:7: error: do not use 'else' after 'return' [readability-else-after-return,-warnings-as-errors]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:818:5: error: use auto when initializing with new to avoid duplicating the type name [modernize-use-auto,-warnings-as-errors]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:825:11: error: variable 'theta2' is not initialized [cppcoreguidelines-init-variables,-warnings-as-errors]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:1422:24: error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:1449:24: error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:1553:24: error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:1577:23: error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:1621:11: error: implicit conversion 'size_type' (aka 'unsigned long') -> 'bool' [readability-implicit-bool-conversion,-warnings-as-errors]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:1621:24: error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]
++ [[ 88 -gt 0 ]]
++ cat /sw/BUILD/11a4e675032c0414b4c52fc6d6c16957d7c44faa/O2Physics-code-check/warnings.txt
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:632:22: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:634:40: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:641:38: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:645:67: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:647:69: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:649:66: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:651:71: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:653:105: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:655:68: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:657:68: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:660:62: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:662:62: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:664:62: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:667:75: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:669:75: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:672:78: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:674:84: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:676:74: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:678:76: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:681:77: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:683:76: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:688:80: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:691:81: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:697:79: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:700:80: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:710:77: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:712:31: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:714:50: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:759:38: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:884:44: warning: statement should be inside braces [readability-braces-around-statements]
[0 more errors; see full log]

Full log here.

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 45b841f at 2026-07-27 19:21:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 1 errors and 0 warnings.'
Found 1 errors and 0 warnings.
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/115b37b7544cf395b96671da584e845b677d635d/O2Physics-code-check/errors.txt
PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx:161:28: error: initializer for member 'ccdbApi' is redundant [readability-redundant-member-init,-warnings-as-errors]
++ [[ 0 -gt 0 ]]
++ [[ 1 -gt 0 ]]
++ exit 1
--

Full log here.

@dmallick2
dmallick2 merged commit f48d2a1 into AliceO2Group:master Jul 27, 2026
14 checks passed
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.

4 participants