Skip to content

[PWGLF] cha01710analysis.cxx: add DCA selection - #17465

Open
jikim1290 wants to merge 8 commits into
AliceO2Group:masterfrom
jikim1290:lf6
Open

[PWGLF] cha01710analysis.cxx: add DCA selection#17465
jikim1290 wants to merge 8 commits into
AliceO2Group:masterfrom
jikim1290:lf6

Conversation

@jikim1290

@jikim1290 jikim1290 commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

add DCA selection

@github-actions github-actions Bot added the pwglf label Aug 14, 2026
@github-actions

github-actions Bot commented Aug 14, 2026

Copy link
Copy Markdown

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

@alibuild

alibuild commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for cffd6d1 at 2026-08-14 16:54:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGLF/Tasks/Resonances/cha01710analysis.cxx:288:37: error: 'cfgV0DaughterDcaPVMin' was not declared in this scope
ninja: build stopped: subcommand failed.

Full log here.

@alibuild

alibuild commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for cffd6d1 at 2026-08-14 16:54:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGLF/Tasks/Resonances/cha01710analysis.cxx:288:37: error: 'cfgV0DaughterDcaPVMin' was not declared in this scope
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 cffd6d1 at 2026-08-14 17:07:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGLF/Tasks/Resonances/cha01710analysis.cxx:288:37: error: 'cfgV0DaughterDcaPVMin' was not declared in this scope
ninja: build stopped: subcommand failed.

Full log here.

@alibuild

alibuild commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for cf42530 at 2026-08-14 22:43:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 3 errors and 2 warnings.'
Found 3 errors and 2 warnings.
++ [[ 3 -gt 0 ]]
++ cat /sw/BUILD/ef20732e93451d5831f5b35ce1f65833b0c04bb9/O2Physics-code-check/errors.txt
PWGLF/Tasks/Resonances/cha01710analysis.cxx:55:8: error: constructor does not initialize these fields: ccdb [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGLF/Tasks/Resonances/cha01710analysis.cxx:275:14: error: redundant boolean literal in conditional return statement [readability-simplify-boolean-expr,-warnings-as-errors]
PWGLF/Tasks/Resonances/cha01710analysis.cxx:388:40: error: repeated branch body in conditional chain [bugprone-branch-clone,-warnings-as-errors]
++ [[ 2 -gt 0 ]]
++ cat /sw/BUILD/ef20732e93451d5831f5b35ce1f65833b0c04bb9/O2Physics-code-check/warnings.txt
PWGLF/Tasks/Resonances/cha01710analysis.cxx:358:31: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/cha01710analysis.cxx:370:69: warning: statement should be inside braces [readability-braces-around-statements]
++ [[ 3 -gt 0 ]]
++ exit 1
--

Full log here.

@alibuild

alibuild commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 2f8d574 at 2026-08-14 23:02:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 3 errors and 2 warnings.'
Found 3 errors and 2 warnings.
++ [[ 3 -gt 0 ]]
++ cat /sw/BUILD/041400feb6a24f08ed5e48167d737a3586c76ae3/O2Physics-code-check/errors.txt
PWGLF/Tasks/Resonances/cha01710analysis.cxx:55:8: error: constructor does not initialize these fields: ccdb [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGLF/Tasks/Resonances/cha01710analysis.cxx:275:14: error: redundant boolean literal in conditional return statement [readability-simplify-boolean-expr,-warnings-as-errors]
PWGLF/Tasks/Resonances/cha01710analysis.cxx:388:40: error: repeated branch body in conditional chain [bugprone-branch-clone,-warnings-as-errors]
++ [[ 2 -gt 0 ]]
++ cat /sw/BUILD/041400feb6a24f08ed5e48167d737a3586c76ae3/O2Physics-code-check/warnings.txt
PWGLF/Tasks/Resonances/cha01710analysis.cxx:358:31: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/cha01710analysis.cxx:370:69: warning: statement should be inside braces [readability-braces-around-statements]
++ [[ 3 -gt 0 ]]
++ exit 1
--

Full log here.

@alibuild alibuild mentioned this pull request Aug 15, 2026
Merged
@alibuild

alibuild commented Aug 15, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 17e06d6 at 2026-08-15 13:02:

## 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/b6e091985db5b8f02ad80957ecefdcc0b9c11113/O2Physics-code-check/errors.txt
PWGLF/Tasks/Resonances/cha01710analysis.cxx:304:7: error: do not use 'else' after 'return' [readability-else-after-return,-warnings-as-errors]
++ [[ 0 -gt 0 ]]
++ [[ 1 -gt 0 ]]
++ exit 1
--

Full log here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alibuild alibuild Awaiting requested review from alibuild alibuild is a code owner
@omvazque omvazque Awaiting requested review from omvazque omvazque is a code owner
@skundu692 skundu692 Awaiting requested review from skundu692 skundu692 is a code owner
@mpuccio mpuccio Awaiting requested review from mpuccio mpuccio is a code owner
@dmallick2 dmallick2 Awaiting requested review from dmallick2 dmallick2 is a code owner
@smaff92 smaff92 Awaiting requested review from smaff92 smaff92 is a code owner
@prottayCMT prottayCMT Awaiting requested review from prottayCMT prottayCMT is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

pwglf

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jikim1290 @alibuild