Skip to content

[PWGJE] Updated GNN b-jet histograms: Resolved issues for merging TH3 histograms on Hyperloop - #17588

Merged
nzardosh merged 3 commits into
AliceO2Group:masterfrom
choich08365:gnnbjet
Aug 25, 2026
Merged

nzardosh merged 3 commits into
AliceO2Group:masterfrom
choich08365:gnnbjet

Conversation

@choich08365

Copy link
Copy Markdown
Contributor

PWGJE/Tasks/bjetTaggingGnn.cxx

  • Reverted TH3 histograms to multiple TH2 histograms as same as before the last update, to resolve bugs occurring when merging TH3 histograms on Hyperloop.
  • Corrected axis titles.

@github-actions github-actions Bot added the pwgje label Aug 25, 2026
@github-actions

Copy link
Copy Markdown

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

@github-actions github-actions Bot changed the title Updated GNN b-jet histograms: Resolved issues for merging TH3 histograms on Hyperloop [PWGJE] Updated GNN b-jet histograms: Resolved issues for merging TH3 histograms on Hyperloop Aug 25, 2026
@choich08365
choich08365 marked this pull request as ready for review August 25, 2026 12:27
@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 0bf6faa at 2026-08-25 15: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/111186a293dc537db9f3cd65fef2aa935dddaadc/O2Physics-code-check/errors.txt
PWGJE/Tasks/bjetTaggingGnn.cxx:46:1: error: included header TH3.h is not used directly [misc-include-cleaner,-warnings-as-errors]
++ [[ 0 -gt 0 ]]
++ [[ 1 -gt 0 ]]
++ exit 1
--

Full log here.

@nzardosh
nzardosh enabled auto-merge (squash) August 25, 2026 13:03
@nzardosh

Copy link
Copy Markdown
Collaborator

Dear @choich08365 after this PR can you make another one to address the code-check errors please

@nzardosh
nzardosh merged commit af3b42e into AliceO2Group:master Aug 25, 2026
14 of 15 checks passed
@choich08365
choich08365 deleted the gnnbjet branch September 8, 2026 01:36
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