Skip to content

[PWGJE] Added analysis code for b-jet tagging using GNN (bjetTaggingGNN.cxx) #9905

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
nzardosh merged 8 commits into from
Feb 18, 2025
Merged

[PWGJE] Added analysis code for b-jet tagging using GNN (bjetTaggingGNN.cxx) #9905

nzardosh merged 8 commits into from
Feb 18, 2025

Conversation

@choich08365
Copy link
Contributor

@choich08365 choich08365 commented Feb 12, 2025 •
edited
Loading

First commit for bjetTaggingGNN.cxx.

  • The task performs b-jet analysis for both MC and data using the GNN b-jet tagging information from o2-analysis-je-jet-taggerhf task.

@github-actions github-actions bot added the pwgje label Feb 12, 2025
@github-actions github-actions bot changed the title Added analysis code for b-jet tagging using GNN (bjetTaggingGNN.cxx) [PWGJE] Added analysis code for b-jet tagging using GNN (bjetTaggingGNN.cxx) Feb 12, 2025
@alibuild alibuild mentioned this pull request Feb 12, 2025
Merged
@choich08365 choich08365 marked this pull request as ready for review February 12, 2025 07:43
@alibuild
Copy link
Collaborator

alibuild commented Feb 12, 2025

Error while checking build/O2Physics/o2 for 0e086fb at 2025-02-12 09:07:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:95:47: error: 'initialiseEventSelection' is not a member of 'jetderiveddatautilities'; did you mean 'initialiseEventSelectionBits'?
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:248:50: error: no matching function for call to 'selectCollision(const o2::soa::Filtered<o2::soa::JoinFull<o2::aod::Hash<1837103245>, o2::soa::JoinFull<o2::aod::Hash<1837103245>, o2::soa::Table<o2::aod::Hash<3292202882>, o2::aod::Hash<1154616615>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2114154707>, o2::aod::Hash<3650294195>, o2::aod::Hash<2286545062> > >, o2::soa::Table<o2::aod::Hash<78943709>, o2::aod::Hash<3105125153>, o2::aod::Hash<2286545062> > > >::iterator&, int&)'
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:307:50: error: no matching function for call to 'selectCollision(const o2::soa::Filtered<o2::soa::JoinFull<o2::aod::Hash<1837103245>, o2::soa::JoinFull<o2::aod::Hash<1837103245>, o2::soa::Table<o2::aod::Hash<3292202882>, o2::aod::Hash<1154616615>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<2114154707>, o2::aod::Hash<3650294195>, o2::aod::Hash<2286545062> > >, o2::soa::Table<o2::aod::Hash<78943709>, o2::aod::Hash<3105125153>, o2::aod::Hash<2286545062> >, o2::soa::Table<o2::aod::Hash<3489904409>, o2::aod::Hash<3504405650>, o2::aod::Hash<2286545062> > > >::iterator&, int&)'
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:197:133: error: unused parameter 'jetFlavor' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:197:155: error: unused parameter 'weight' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:214:135: error: unused parameter 'jetFlavor' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:214:157: error: unused parameter 'weight' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:197:133: error: unused parameter 'jetFlavor' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:197:155: error: unused parameter 'weight' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:214:135: error: unused parameter 'jetFlavor' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:214:157: error: unused parameter 'weight' [-Werror=unused-parameter]
ninja: build stopped: subcommand failed.

Full log here.

@alibuild
Copy link
Collaborator

alibuild commented Feb 12, 2025

Error while checking build/O2Physics/o2 for 443a743 at 2025-02-12 09:46:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:197:133: error: unused parameter 'jetFlavor' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:197:155: error: unused parameter 'weight' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:214:135: error: unused parameter 'jetFlavor' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:214:157: error: unused parameter 'weight' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:197:133: error: unused parameter 'jetFlavor' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:197:155: error: unused parameter 'weight' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:214:135: error: unused parameter 'jetFlavor' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/9905-slc9_x86-64/0/PWGJE/Tasks/bjetTaggingGNN.cxx:214:157: error: unused parameter 'weight' [-Werror=unused-parameter]
ninja: build stopped: subcommand failed.

Full log here.

@alibuild alibuild mentioned this pull request Feb 12, 2025
Merged
@choich08365
Merge pull request #3 from alibuild/alibot-cleanup-9905
27f3072 
Please consider the following formatting changes to AliceO2Group#9905
@nzardosh
Copy link
Collaborator

nzardosh commented Feb 13, 2025

@hahassan7 can you please make sure this is consistent with the framework you have put in place?

@hahassan7
Copy link
Collaborator

hahassan7 commented Feb 18, 2025

Hi @nzardosh , Sorry for my late reply. I didn't get any notification from here. I just checked the code and it seems good to go.

@nzardosh nzardosh merged commit 0d20013 into AliceO2Group:master Feb 18, 2025
11 of 13 checks passed
alibuild added a commit to alibuild/O2Physics that referenced this pull request Aug 11, 2025
@choich08365 @alibuild @CesarOmarRA
[PWGJE] Added analysis code for b-jet tagging using GNN (bjetTaggingG…
3a76794 
…NN.cxx) (AliceO2Group#9905)

Co-authored-by: Changhwan Choi <changhwan.choi@cern.ch>
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nzardosh nzardosh nzardosh approved these changes

@alibuild alibuild Awaiting requested review from alibuild alibuild is a code owner

@maoyx maoyx Awaiting requested review from maoyx maoyx is a code owner

@fjonasALICE fjonasALICE Awaiting requested review from fjonasALICE fjonasALICE is a code owner

@mfasDa mfasDa Awaiting requested review from mfasDa mfasDa is a code owner

@mhemmer-cern mhemmer-cern Awaiting requested review from mhemmer-cern mhemmer-cern is a code owner

Assignees

No one assigned

Labels

pwgje

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@choich08365 @alibuild @nzardosh @hahassan7