search

vhbb / cmssw

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

Fix ele id for zee vhbb heppy80 x #663

Closed degrutto closed 7 years ago

degrutto commented 7 years ago

remove hlt-like offline id cut for Zee, as they are not needed in case we use double electron. Running on 1K event ZH sample, you can see how Vtype changes before

http://degrutto.web.cern.ch/degrutto/2017/before.pdf

and after this change

http://degrutto.web.cern.ch/degrutto/2017/after.pdf

as you can see the Vtype=1 bin content goes from 69 -> 76, so this is not a small effect

arizzi commented 7 years ago

a bit late, isn't it? vtype flag is the key to avoid double counting, what's the plan now? you plan to rewrite the Vtype logic in the analysis code? if so you need to implement THE EXACT same code in all analysis otherwise we double count!

degrutto commented 7 years ago

Hi

Now this in merged we should also fix the swap id80 -> 90 for Zee and wen

Sent from my iPhone

On 13 Feb 2017, at 08:30, arizzi notifications@github.com wrote:

Merged #663.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

arizzi commented 7 years ago

did already in my local area, I will commit