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

Updated MVA lepton IDs #651

Open
wants to merge 6 commits into
base: vhbbHeppy80X
Choose a base branch
from

Conversation

veelken
Copy link

@veelken veelken commented Jan 27, 2017

  • added HZZ electron ID from EGamma POG
  • updated MVA lepton ID developed by ttH multilepton analysis

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
Copy link

arizzi commented Jan 27, 2017

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
Copy link
Author

veelken commented Jan 27, 2017

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

@arizzi arizzi added the Heppy label Jan 28, 2017
@degrutto
Copy link

Hi @veelken

here in this 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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants