search

cms-nanoAOD / cmssw

CMS NanoAOD software integration repository
http://cms-sw.github.io/
Apache License 2.0
3 stars 10 forks source link

Fall17 v2 electron id for era 94X2016 #272

Closed mgratti closed 5 years ago

mgratti commented 5 years ago

Hi, sorry to come back to this, but it looks like recommendations for electron ids are evolving and there is a concrete possibility that Fall17 v2 is going to be recommended also for 2016. Can I add the vid nested bitmap for Fall17 v2 electron id for 94X2016?

peruzzim commented 5 years ago

please see #267 (to be fixed because it does not run at the moment)

should be discussed how widely and strongly this is recommended by EGM because at the moment it's not on track to make it to the first production @michelif @Sam-Harper @lsoffi @swagata87

lsoffi commented 5 years ago

Hi, as we have stated today [https://indico.cern.ch/event/777545/contributions/3234587/attachments/1766655/2868714/micheli_EG_cmsweek_20181206_4.pdf] Fall17V2 is what EGM recommend to use also for 2016 if the analysis is starting now. If the analysis is already ongoing using 2016 80X ID one is still fine. So things are not evolving. I hope this make it clear how widely and strongly EGM recommend this ID.

Sam-Harper commented 5 years ago

please see #267 (to be fixed because it does not run at the moment)

should be discussed how widely and strongly this is recommended by EGM because at the moment it's not on track to make it to the first production @michelif @Sam-Harper @lsoffi @swagata87

Its fixed. Sorry, too many balls in the air

Sam-Harper commented 5 years ago

Now pushed. Now on this, as Livia says we would like this. However we are late and the nano can not wait indefinitely so if it misses the deadline, we can put in the next itteration.

peruzzim commented 5 years ago

Ok, thanks. We will try to make it happen profiting from the fact that a central validation round has still to take place, and production should start with 2017 that is not affected by this.

peruzzim commented 5 years ago

requested to be queued for integration here https://github.com/cms-sw/cmssw/issues/25425#issuecomment-444830591

Sam-Harper commented 5 years ago

thanks Marco. Again we dont want to be unreasonable here though so if we're too slow, well thats our fault and the train leaves without us