Skip to content

[PWGJE] Add matched proton and pair efficiencies - #18294

Merged
nzardosh merged 4 commits into
AliceO2Group:masterfrom
hanseopark:corrInJet
Oct 16, 2026
Merged

nzardosh merged 4 commits into
AliceO2Group:masterfrom
hanseopark:corrInJet

Conversation

@hanseopark

Copy link
Copy Markdown
Contributor
  • Adds charged-jet response and matching QA.
  • Adds single-proton tracking, TOF-matching, PID, and total-efficiency histograms.
  • Adds same-sign and opposite-sign proton-pair efficiency histograms.
  • Fixes MCD track-table binding and removes the unnecessary multiplicity dependency from inclusive same-event processing.
  • Fixes include-cleaner and clang-tidy issues.

@github-actions

Copy link
Copy Markdown

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

nzardosh
nzardosh previously approved these changes Oct 14, 2026
@nzardosh
nzardosh enabled auto-merge (squash) October 14, 2026 15:11
@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for b60182f at 2026-10-14 18:56:

## 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 3 warnings.'
Found 1 errors and 3 warnings.
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/52482b538b20d4b75a40b2412eecacfe99b3fda5/O2Physics-code-check/errors.txt
PWGJE/Tasks/baryonPairInJets.cxx:20:1: error: included header EventSelectionParams.h is not used directly [misc-include-cleaner,-warnings-as-errors]
++ [[ 3 -gt 0 ]]
++ cat /sw/BUILD/52482b538b20d4b75a40b2412eecacfe99b3fda5/O2Physics-code-check/warnings.txt
PWGJE/Tasks/baryonPairInJets.cxx:1554:39: warning: redundant explicit casting to the same type 'int64_t' (aka 'long') as the sub-expression, remove this casting [readability-redundant-casting]
PWGJE/Tasks/baryonPairInJets.cxx:1570:35: warning: redundant explicit casting to the same type 'int64_t' (aka 'long') as the sub-expression, remove this casting [readability-redundant-casting]
PWGJE/Tasks/baryonPairInJets.cxx:1581:50: warning: use designated initializer list to initialize 'SelectedParticle' [modernize-use-designated-initializers]
++ [[ 1 -gt 0 ]]
++ exit 1
--

Full log here.

auto-merge was automatically disabled October 15, 2026 05:52

Head branch was pushed to by a user without write access

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for a82c783 at 2026-10-15 08:45:

No log files found

Full log here.

@hanseopark

Copy link
Copy Markdown
Contributor Author

@nzardosh All checks now pass on the latest commit (13/13). Could you please re-enable auto-merge when convenient?

@nzardosh
nzardosh merged commit d2eec1a into AliceO2Group:master Oct 16, 2026
13 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