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
Pileup jet ID for JPT #1872
Pileup jet ID for JPT #1872
Conversation
A new Pull Request was created by @rkogler (Roman Kogler) for CMSSW_7_0_X. Pileup jet ID for JPT It involves the following packages: RecoJets/JetProducers @nclopezo, @cmsbuild, @anton-a, @thspeer, @slava77 can you please review it and eventually sign? Thanks. |
// | ||
// Original Author: Olga Kodolova,40 R-A12,+41227671273, | ||
// Created: Fri Jan 11 15:30:47 CEST 2013 | ||
// $Id: PileupJPTJetIdProducer.cc,v 1.1 2013/03/27 17:42:09 rkogler Exp $ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove CVS tags
|
||
// ------------------------------------------------------------------------------------------ | ||
void | ||
PileupJPTJetIdProducer::beginRun(edm::Run&, edm::EventSetup const&) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove begin/end Run/Job from .h and .cc , since they are empty
+1 |
This pull request is fully signed and it will be integrated in one of the next IBs unless changes (tests are also fine). @ktf can you please take care of it? |
Reco update -- Pileup jet ID for JPT
Pileup jet ID for JPT jets included. This update also includes weight files for the standard pileup jet ID, which are now stored in an external package (and downloaded through download.url).