Skip to content

[PWGJE] Updated GNN b-jet analysis tasks - #17506

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

[PWGJE] Updated GNN b-jet analysis tasks#17506
choich08365 wants to merge 4 commits into
AliceO2Group:masterfrom
choich08365:gnnbjet

Conversation

@choich08365

Copy link
Copy Markdown
Contributor

PWGJE/Tasks/bjetTaggingGnn.cxx

  • Refactored process functions using templates.
  • Updated processes for UE subtraction and SV mass template fit.
  • Applied event selection flags for OO collisions.

PWGJE/Tasks/bjetTreeCreator.cxx

  • Minor revisions: comments for future discussion on using PID info for training the GNN model, track DCA selection criteria matched to main analysis task.

@github-actions github-actions Bot added the pwgje label Aug 19, 2026
@github-actions github-actions Bot changed the title Updated GNN b-jet analysis tasks [PWGJE] Updated GNN b-jet analysis tasks Aug 19, 2026
@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown

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

@choich08365
choich08365 marked this pull request as ready for review August 19, 2026 09:28
@alibuild

alibuild commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 07ae072 at 2026-08-20 07:41:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 9 errors and 21 warnings.'
Found 9 errors and 21 warnings.
++ [[ 9 -gt 0 ]]
++ cat /sw/BUILD/a6cccc0d80164103059dfd8d0339b8945edee378/O2Physics-code-check/errors.txt
PWGJE/Tasks/bjetTaggingGnn.cxx:50:1: error: included header type_traits is not used directly [misc-include-cleaner,-warnings-as-errors]
PWGJE/Tasks/bjetTaggingGnn.cxx:166:31: error: no header providing "TAxis" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGJE/Tasks/bjetTaggingGnn.cxx:178:8: error: constructor does not initialize these fields: pdg, ccdb [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGJE/Tasks/bjetTaggingGnn.cxx:632:85: error: no header providing "o2::aod::BkgChargedRhos" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGJE/Tasks/bjetTaggingGnn.cxx:660:92: error: no header providing "o2::aod::BkgChargedMcRhos" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGJE/Tasks/bjetTaggingGnn.cxx:700:15: error: member 'largeNegativeNumber' of type 'const float' is const qualified [cppcoreguidelines-avoid-const-or-ref-data-members,-warnings-as-errors]
PWGJE/Tasks/bjetTaggingGnn.cxx:701:15: error: member 'largePositiveNumber' of type 'const float' is const qualified [cppcoreguidelines-avoid-const-or-ref-data-members,-warnings-as-errors]
PWGJE/Tasks/bjetTaggingGnn.cxx:1084:13: error: member 'nPartInel0' of type 'const int' is const qualified [cppcoreguidelines-avoid-const-or-ref-data-members,-warnings-as-errors]
PWGJE/Tasks/bjetTaggingGnn.cxx:1456:14: error: variable 'jetFlavor' is not initialized [cppcoreguidelines-init-variables,-warnings-as-errors]
++ [[ 21 -gt 0 ]]
++ cat /sw/BUILD/a6cccc0d80164103059dfd8d0339b8945edee378/O2Physics-code-check/warnings.txt
PWGJE/Tasks/bjetTaggingGnn.cxx:55:10: warning: inclusion of deprecated C++ header 'math.h'; consider using 'cmath' instead [modernize-deprecated-headers]
PWGJE/Tasks/bjetTaggingGnn.cxx:145:54: warning: use designated initializer list to initialize 'EvtSelStage' [modernize-use-designated-initializers]
PWGJE/Tasks/bjetTaggingGnn.cxx:146:54: warning: use designated initializer list to initialize 'EvtSelStage' [modernize-use-designated-initializers]
PWGJE/Tasks/bjetTaggingGnn.cxx:147:54: warning: use designated initializer list to initialize 'EvtSelStage' [modernize-use-designated-initializers]
PWGJE/Tasks/bjetTaggingGnn.cxx:148:54: warning: use designated initializer list to initialize 'EvtSelStage' [modernize-use-designated-initializers]
PWGJE/Tasks/bjetTaggingGnn.cxx:149:54: warning: use designated initializer list to initialize 'EvtSelStage' [modernize-use-designated-initializers]
PWGJE/Tasks/bjetTaggingGnn.cxx:150:54: warning: use designated initializer list to initialize 'EvtSelStage' [modernize-use-designated-initializers]
PWGJE/Tasks/bjetTaggingGnn.cxx:151:54: warning: use designated initializer list to initialize 'EvtSelStage' [modernize-use-designated-initializers]
PWGJE/Tasks/bjetTaggingGnn.cxx:152:54: warning: use designated initializer list to initialize 'EvtSelStage' [modernize-use-designated-initializers]
PWGJE/Tasks/bjetTaggingGnn.cxx:153:54: warning: use designated initializer list to initialize 'EvtSelStage' [modernize-use-designated-initializers]
PWGJE/Tasks/bjetTaggingGnn.cxx:154:54: warning: use designated initializer list to initialize 'EvtSelStage' [modernize-use-designated-initializers]
PWGJE/Tasks/bjetTaggingGnn.cxx:155:54: warning: use designated initializer list to initialize 'EvtSelStage' [modernize-use-designated-initializers]
PWGJE/Tasks/bjetTaggingGnn.cxx:156:54: warning: use designated initializer list to initialize 'EvtSelStage' [modernize-use-designated-initializers]
PWGJE/Tasks/bjetTaggingGnn.cxx:157:54: warning: use designated initializer list to initialize 'EvtSelStage' [modernize-use-designated-initializers]
PWGJE/Tasks/bjetTaggingGnn.cxx:158:54: warning: use designated initializer list to initialize 'EvtSelStage' [modernize-use-designated-initializers]
PWGJE/Tasks/bjetTaggingGnn.cxx:1089:43: warning: statement should be inside braces [readability-braces-around-statements]
PWGJE/Tasks/bjetTaggingGnn.cxx:1094:46: warning: statement should be inside braces [readability-braces-around-statements]
PWGJE/Tasks/bjetTaggingGnn.cxx:1533:34: warning: use 'contains' to check for membership [readability-container-contains]
PWGJE/Tasks/bjetTaggingGnn.cxx:1545:34: warning: use 'contains' to check for membership [readability-container-contains]
PWGJE/Tasks/bjetTreeCreator.cxx:52:10: warning: inclusion of deprecated C++ header 'math.h'; consider using 'cmath' instead [modernize-deprecated-headers]
PWGJE/Tasks/bjetTreeCreator.cxx:814:9: warning: implicit conversion 'int' -> 'bool' [readability-implicit-bool-conversion]
++ [[ 9 -gt 0 ]]
++ exit 1
--

Full log here.

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.

2 participants