CERN-PH-CMG / cmg-cmssw

CERN CMG Tools repository. Installation instructions available on the twiki:
https://twiki.cern.ch/twiki/bin/viewauth/CMS/CMGToolsReleasesExperimental
19 stars 244 forks source link

Heppy updates: 2017 tau variables and features sync to nanoAOD #736

Closed peruzzim closed 6 years ago

peruzzim commented 6 years ago

Summary of changes:

gpetruc commented 6 years ago

Looks ok to me. @steggema any opinion on taus?

steggema commented 6 years ago

737 has a few more updates that will also allow to evaluate the MVA ID working points, but this should merge graciously since the #737 is based on the developments/commits included here so I think the MVA ID part is OK.

The other change is certainly fine if the goal is to have the same information as in NanoAOD - one would typically still apply an additional cut to make sure that the tau vertex corresponds to the primary vertex, and I'm not sure how this is solved in the TTH code.

peruzzim commented 6 years ago

This code is to get the value of the dxy with respect to the PV, a selection is later required on this value as a part of the standard tau ID requirements.