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
Updates for PAT python configuration files #3510
Updates for PAT python configuration files #3510
Conversation
…e it has been disabled
like pfElectronsFromVertex and pfMuonsFromVertex), also turned it into a more generic tool
A new Pull Request was created by @ferencek (Dinko Ferencek) for CMSSW_7_1_X. Updates for PAT python configuration files It involves the following packages: PhysicsTools/PatAlgos @nclopezo, @vadler, @cmsbuild, @Degano, @monttj can you please review it and eventually sign? Thanks. |
@ferencek Wouldn't it be better to make JME default so that endusers do not need to change naming in their configuration file? Then we need to add the option to check closest Z vertex for pfNoPileUpIso. |
@monttj That is not my call. I simply fixed the module names as defined in https://github.com/cms-sw/cmssw/tree/CMSSW_7_1_X/CommonTools/ParticleFlow/python Note that there are now 3 pfNoPileUp sequences defined (pfNoPileUp, pfNoPileUpIso, and pfNoPileUpJME) |
+1 |
…rom-CMSSW_7_1_X_2014-04-25-1400 AT -- Updates for PAT python configuration files
This PR is the second part of the closed #3408 after factorization. It primarily addresses some of the problems reported in https://hypernews.cern.ch/HyperNews/CMS/get/physTools/3137.html