Skip to content

Conversation

@ariedel-cern
Copy link
Collaborator

In this PR:

  • fix bug in filling of MC labels (now using only one map to keep track of filled MC particles instead of dedicated maps for all particle species)
  • add MC processing in generic pairHistManager
  • add MC proccessing to track-track, track-v0 and track-kink tasks
  • add option for common/non-common cut to pairCleaner via pairCuts configurable

@github-actions
Copy link

github-actions bot commented Jan 2, 2026

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

alibuild
alibuild previously approved these changes Jan 2, 2026
Copy link
Collaborator

@alibuild alibuild left a comment

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 @ariedel-cern.

@ariedel-cern
Copy link
Collaborator Author

MegaLinter flags python script because it cannot import ROOT. Script works fine. Ignoring.

@alibuild
Copy link
Collaborator

alibuild commented Jan 2, 2026

Error while checking build/O2Physics/o2 for ba5b4d1 at 2026-01-02 20:27:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/14373-slc9_x86-64/0/PWGCF/Femto/Core/mcBuilder.h:263:12: error: variable 'mcParticle' set but not used [-Werror=unused-but-set-variable]
/sw/SOURCES/O2Physics/14373-slc9_x86-64/0/PWGCF/Femto/Core/mcBuilder.h:263:12: error: variable 'mcParticle' set but not used [-Werror=unused-but-set-variable]
/sw/SOURCES/O2Physics/14373-slc9_x86-64/0/PWGCF/Femto/Core/mcBuilder.h:263:12: error: variable 'mcParticle' set but not used [-Werror=unused-but-set-variable]
/sw/SOURCES/O2Physics/14373-slc9_x86-64/0/PWGCF/Femto/Core/mcBuilder.h:263:12: error: variable 'mcParticle' set but not used [-Werror=unused-but-set-variable]
/sw/SOURCES/O2Physics/14373-slc9_x86-64/0/PWGCF/Femto/Core/mcBuilder.h:263:12: error: variable 'mcParticle' set but not used [-Werror=unused-but-set-variable]
ninja: build stopped: subcommand failed.

Full log here.

Copy link
Collaborator

@alibuild alibuild left a comment

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 @ariedel-cern.

@ariedel-cern ariedel-cern merged commit 4bb7594 into AliceO2Group:master Jan 2, 2026
12 of 13 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