Skip to content

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

Open
choich08365 wants to merge 3 commits into
AliceO2Group:masterfrom
choich08365:gnnbjet
Open

[PWGJE] Updated GNN b-jet histograms: Resolved issues for merging TH3 histograms on Hyperloop#17588
choich08365 wants to merge 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

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