Skip to content

[PWGCF] FU D0 task: bug fixing, unused elements removal, new TrackQA plots - #18498

Merged
prchakra merged 2 commits into
AliceO2Group:masterfrom
kgwizdzi:femtoUniD0Task20261028
Oct 29, 2026
Merged

prchakra merged 2 commits into
AliceO2Group:masterfrom
kgwizdzi:femtoUniD0Task20261028

Conversation

@kgwizdzi

Copy link
Copy Markdown
Contributor

Changes in the code:

  • functions getDeltaPhi and wrapDeltaPhi0PI not used and removed
  • unused configurables removed
  • partitions with D0 and D0bar candidates - naming correction
  • partitions with D0 and D0bar candidates which pass only one mass hypothesis removed - not used (Note: maybe in the future if it will be need then other solution can be applied in the code than partitions)
  • new QA plots for tracks added (They are filled in the processPurityQA function if they are enabled with fillTrackQA configurable)
  • processQAD0D0barSel removed - not used
  • new process functions for SE and ME: processSameEventSigD0, processSameEventSigD0bar, processSameEventSideBandD0, processSameEventSideBandD0bar, processMixedEventSigD0, processMixedEventSigD0bar (Note: after cross-check with the results from the current functions like processSameEventWithDoubleHypo and processSameEventSB the old ones will be removed and configurable used to switch between D0 and D0bar candidates)

@github-actions github-actions Bot added the pwgcf label Oct 28, 2026
@github-actions github-actions Bot changed the title FU D0 task: bug fixing, unused elements removal, new TrackQA plots [PWGCF] FU D0 task: bug fixing, unused elements removal, new TrackQA plots Oct 28, 2026
@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 dd947bc at 2026-10-29 00:53:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'To reproduce the results below locally, run: '\''aliBuild build O2Physics-code-check -e ALIBUILD_BASE_HASH=upstream/master'\''.'
To reproduce the results below locally, run: 'aliBuild build O2Physics-code-check -e ALIBUILD_BASE_HASH=upstream/master'.
++ echo 'To apply all automatic fixes, add the '\''-e O2PHYSICS_CHECKER_FIX=1'\'' option.'
To apply all automatic fixes, add the '-e O2PHYSICS_CHECKER_FIX=1' option.
++ echo 'See the documentation for more details at: <https://aliceo2group.github.io/analysis-framework/docs/tools/#running-with-o2physics-code-check>.'
See the documentation for more details at: <https://aliceo2group.github.io/analysis-framework/docs/tools/#running-with-o2physics-code-check>.
++ echo 'Found 1 errors and 0 warnings.'
Found 1 errors and 0 warnings.
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/35f40dc44b6695bfaefa4957a725cffaac53a441/O2Physics-code-check/errors.txt
PWGCF/FemtoUniverse/Tasks/femtoUniversePairTaskTrackD0.cxx:28:1: error: included header RecoDecay.h is not used directly [misc-include-cleaner,-warnings-as-errors]
++ [[ 0 -gt 0 ]]
++ [[ 1 -gt 0 ]]
++ exit 1
--

Full log here.

@prchakra
prchakra merged commit b0bddaa into AliceO2Group:master Oct 29, 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.

3 participants