search

vhbb / cmssw

CMS Offline Software
cms-sw.github.io/cmssw
4 stars 5 forks source link

double-b retraining #585

Closed cvernier closed 7 years ago

cvernier commented 7 years ago

new training for double-b needed for Summer-16

arizzi commented 7 years ago

which samples? how many events?

cvernier commented 7 years ago

just need the recipe to include the latest training. https://twiki.cern.ch/twiki/bin/view/CMS/Hbbtagging#V4_training

arizzi commented 7 years ago

how about this:

[image: Warning, important] Currently the V4 training is only provided for AK8 jets.

On Mon, Jan 9, 2017 at 2:20 PM, Caterina Vernieri notifications@github.com wrote:

just need to inclute the recipe to include the latest training. https://twiki.cern.ch/twiki/bin/view/CMS/Hbbtagging#V4_training

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/vhbb/cmssw/issues/585#issuecomment-271282323, or mute the thread https://github.com/notifications/unsubscribe-auth/AEyilt3QjISMl1RuNuL2B99wsRThpJLLks5rQjQBgaJpZM4LPh_C .

cvernier commented 7 years ago

it's ok, as far as I know no analysis is using the CA15 jets. the old one is still valid for CA15.

arizzi commented 7 years ago

we do run CA15... are we really not using it @gkasieczka @ferencek ?

ferencek commented 7 years ago

Ideally, it would be nice if we had v4 training for CA15 jets as well. But since we don't, we'll have to stay with v3 for CA15 and switch to v4 for AK8. Note that the training files are explicitly defined in https://github.com/vhbb/cmssw/blob/vhbbHeppy80X/VHbbAnalysis/Heppy/test/combined_cmssw.py#L332-L344 so that needs to be updated in addition to following the recipe at https://twiki.cern.ch/twiki/bin/viewauth/CMS/Hbbtagging#V4_training

I also noticed that the input candidates for IPTagInfos in https://github.com/vhbb/cmssw/blob/vhbbHeppy80X/VHbbAnalysis/Heppy/test/combined_cmssw.py#L361 are incorrectly set (they should be set to the full collection of PF candidates, i.e. particleFlow for AOD and packedPFCandidates for MiniAOD).

ferencek commented 7 years ago

In addition, I guess https://github.com/vhbb/cmssw/blob/vhbbHeppy80X/VHbbAnalysis/Heppy/test/combined_cmssw.py#L671-L687 can be removed or commented out.

arizzi commented 7 years ago

it should be in fact converted into running deepCSV

cvernier commented 7 years ago

602