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
Modernize GoodVetexFilter #10794
Modernize GoodVetexFilter #10794
Conversation
Added necessary consumes call, switched to getByToken and converted to be a 'global' module.
@slava77 This should fix the problem with GoodVertexFilter |
A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_7_6_X. Modernize GoodVetexFilter It involves the following packages: DPGAnalysis/Skims @cmsbuild, @srimanob, @boudoul, @franzoni can you please review it and eventually sign? Thanks. |
please test |
@davidlange6 in case you could merge that before -2300 build starts, it would be nice |
The tests are being triggered in jenkins. |
+1 |
Added necessary consumes call, switched to getByToken and converted
to be a 'global' module.