search

vhbb / cmssw

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

Updated MVA lepton IDs #651

Open veelken opened 7 years ago

veelken commented 7 years ago

NOTE: the HZZ electron ID requires to pull code from the git repository of the EGamma POG:

cmsrel CMSSW_8_0_21 cd CMSSW_8_0_21/src eval scramv1 runtime -sh git cms-merge-topic ikrav:egm_id_80X_v2 git cms-merge-topic -u vhbb:vhbbHeppy80X cd $CMSSW_BASE/external cd slc6_amd64_gcc530/ git clone https://github.com/ikrav/RecoEgamma-ElectronIdentification.git data/RecoEgamma/ElectronIdentification/data cd data/RecoEgamma/ElectronIdentification/data git checkout egm_id_80X_v1 cd $CMSSW_BASE/src scram b -j 16

arizzi commented 7 years ago

how urgent is this stuff? we are about to push a button and the amount of stuff included here is not negligible possibly requiring another round of validation... (PS: are you subscribed to the vhbb heppy egroup?!?)

veelken commented 7 years ago

Hi Andrea,

I would appreciate if you merge this PR as soon as possible, but I understand that you are about to start the production. It is fine for me if you the PR after starting the production (people working on ttH, H->tautau need to run a separate VHbb Ntuple production without the Njet >= 2 cut anyway).

Cheers,

Christian

P.S. I am subscribed to the vhbb heppy egroup

degrutto commented 7 years ago

Hi @veelken

here in this issues https://github.com/vhbb/cmssw/issues/652 we were also wondering if we better do not need to apply the ele_mvaEleID_Trig_preselection(x)

to the Z-> ee electrons in case we use double electron triggers, what it is your opinion?