Skip to content
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.

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

Customisation functions for jet tagging #64

Merged
merged 1 commit into from
Jun 8, 2023
Merged

Conversation

thesps
Copy link
Contributor

@thesps thesps commented Jun 6, 2023

Following from the recent discussions on centralising some jet tagging ntupling, here I add the methods to save the content we used for some previous studies.

Add variables from jet constituents

  • pt
  • eta
  • phi
  • mass
  • pdgId

Save GenJet hadron&parton-based flavour (https://twiki.cern.ch/twiki/bin/view/CMSPublic/SWGuideBTagMCTools#Hadron_parton_based_jet_flavour)

…constituents, save GenJet hardon&parton-based flavour
@jmduarte
Copy link
Member

jmduarte commented Jun 6, 2023

@thesps this is great!

I was trying to do something similar, but my approach was going to be to save all the particle candidates using saveCands (like we do for ML MET training), but adding an index to tell it which jet it belongs to.

This has the benefit that you can save all the candidates, but then you have to do some postprocessing to make the training arrays.

@violatingcp @ddiaz006

@artlbv
Copy link

artlbv commented Jun 7, 2023

hi folks!
@thesps does the output ntuple you are pushing here correspond to what we have been using for the jet ID studies last year?
I was also wondering what ntuples the b-tag team @violatingcp was using if not this repo?

@jmduarte would it be easy to create training datasets with the format single jet+constituents from the files you were using?
The advantage of the method Sioni uses is that this is much closer to what the FW will be doing, right?

@artlbv
Copy link

artlbv commented Jun 7, 2023

maybe I can add @EmyrClement here to the thread

@gpetruc gpetruc merged commit adece56 into 12_5_X Jun 8, 2023
@thesps thesps deleted the for_jet_tagging branch November 14, 2024 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants