diff --git a/PhysicsTools/PatAlgos/plugins/JetCorrFactorsProducer.cc b/PhysicsTools/PatAlgos/plugins/JetCorrFactorsProducer.cc index 168329e6264b6..668d9c06607ca 100644 --- a/PhysicsTools/PatAlgos/plugins/JetCorrFactorsProducer.cc +++ b/PhysicsTools/PatAlgos/plugins/JetCorrFactorsProducer.cc @@ -174,32 +174,15 @@ JetCorrFactorsProducer::JetCorrFactorsProducer(const edm::ParameterSet& cfg) // L1Offset correction, which is an additional input to the L1JPTOffset // corrector if (std::find(levels.begin(), levels.end(), "L1JPTOffset") != levels.end()) { - if (cfg.existsAs("extraJPTOffset")) { - extraJPTOffset_.push_back(cfg.getParameter("extraJPTOffset")); - } else { - throw cms::Exception("No parameter extraJPTOffset specified") - << "The configured correction levels contain a L1JPTOffset correction, which re- \n" - << "quires the additional parameter extraJPTOffset or type std::string. This \n" - << "string should correspond to the L1Offset corrections that should be applied \n" - << "together with the JPTL1Offset corrections. These corrections can be of type \n" - << "L1Offset or L1FastJet. \n"; - } + extraJPTOffset_.push_back(cfg.getParameter("extraJPTOffset")); } // if the std::string L1Offset can be found in levels an additional para- // meter primaryVertices is needed, which should pass on the offline pri- // mary vertex collection. The size of this collection is needed for the // L1Offset correction. if (useNPV_) { - if (cfg.existsAs("primaryVertices")) { - primaryVertices_ = cfg.getParameter("primaryVertices"); - primaryVerticesToken_ = mayConsume >(primaryVertices_); - } else { - throw cms::Exception("No primaryVertices specified") - << "The configured correction levels contain an L1Offset or L1FastJet correction, \n" - << "which requires the number of offlinePrimaryVertices. Please specify this col- \n" - << "lection as additional optional parameter primaryVertices of type edm::InputTag\n" - << "in the jetCorrFactors module. \n"; - } + primaryVertices_ = cfg.getParameter("primaryVertices"); + primaryVerticesToken_ = mayConsume >(primaryVertices_); } // if the std::string L1FastJet can be found in levels an additional // parameter rho is needed, which should pass on the energy density @@ -207,16 +190,8 @@ JetCorrFactorsProducer::JetCorrFactorsProducer(const edm::ParameterSet& cfg) if (useRho_) { if ((!extraJPTOffset_.empty() && extraJPTOffset_.front() == std::string("L1FastJet")) || std::find(levels.begin(), levels.end(), "L1FastJet") != levels.end()) { - if (cfg.existsAs("rho")) { - rho_ = cfg.getParameter("rho"); - rhoToken_ = mayConsume(rho_); - } else { - throw cms::Exception("No parameter rho specified") - << "The configured correction levels contain a L1FastJet correction, which re- \n" - << "quires the energy density parameter rho. Please specify this collection as \n" - << "additional optional parameter rho of type edm::InputTag in the jetCorrFac- \n" - << "tors module. \n"; - } + rho_ = cfg.getParameter("rho"); + rhoToken_ = mayConsume(rho_); } else { edm::LogInfo message("Parameter rho not used"); message << "Module is configured to use the parameter rho, but rho is only used \n" diff --git a/PhysicsTools/PatAlgos/plugins/PATElectronProducer.cc b/PhysicsTools/PatAlgos/plugins/PATElectronProducer.cc index 7601f81769617..303f2195f214b 100644 --- a/PhysicsTools/PatAlgos/plugins/PATElectronProducer.cc +++ b/PhysicsTools/PatAlgos/plugins/PATElectronProducer.cc @@ -316,10 +316,7 @@ PATElectronProducer::PATElectronProducer(const edm::ParameterSet& iConfig) pvToken_(mayConsume>(iConfig.getParameter("pvSrc"))), addElecID_(iConfig.getParameter("addElectronID")), pTComparator_(), - isolator_(iConfig.exists("userIsolation") ? iConfig.getParameter("userIsolation") - : edm::ParameterSet(), - consumesCollector(), - false), + isolator_(iConfig.getParameter("userIsolation"), consumesCollector(), false), addEfficiencies_(iConfig.getParameter("addEfficiencies")), addResolutions_(iConfig.getParameter("addResolutions")), useUserData_(iConfig.exists("userData")), @@ -328,14 +325,8 @@ PATElectronProducer::PATElectronProducer(const edm::ParameterSet& iConfig) // MC matching configurables (scheduled mode) if (addGenMatch_) { - if (iConfig.existsAs("genParticleMatch")) { - genMatchTokens_.push_back(consumes>( - iConfig.getParameter("genParticleMatch"))); - } else { - genMatchTokens_ = edm::vector_transform( - iConfig.getParameter>("genParticleMatch"), - [this](edm::InputTag const& tag) { return consumes>(tag); }); - } + genMatchTokens_.push_back(consumes>( + iConfig.getParameter("genParticleMatch"))); } // resolution configurables if (addResolutions_) { diff --git a/PhysicsTools/PatAlgos/plugins/PATMuonProducer.cc b/PhysicsTools/PatAlgos/plugins/PATMuonProducer.cc index 59e540d633109..50605edbd950a 100644 --- a/PhysicsTools/PatAlgos/plugins/PATMuonProducer.cc +++ b/PhysicsTools/PatAlgos/plugins/PATMuonProducer.cc @@ -352,10 +352,7 @@ PATMuonProducer::PATMuonProducer(const edm::ParameterSet& iConfig, PATMuonHeavyO computeSoftMuonMVA_(false), recomputeBasicSelectors_(false), mvaUseJec_(false), - isolator_(iConfig.exists("userIsolation") ? iConfig.getParameter("userIsolation") - : edm::ParameterSet(), - consumesCollector(), - false), + isolator_(iConfig.getParameter("userIsolation"), consumesCollector(), false), geometryToken_{esConsumes()}, transientTrackBuilderToken_{esConsumes(edm::ESInputTag("", "TransientTrackBuilder"))}, patMuonPutToken_{produces>()} { diff --git a/PhysicsTools/PatAlgos/plugins/PATPhotonProducer.cc b/PhysicsTools/PatAlgos/plugins/PATPhotonProducer.cc index e44054c58a6a0..46deba6ff8be8 100644 --- a/PhysicsTools/PatAlgos/plugins/PATPhotonProducer.cc +++ b/PhysicsTools/PatAlgos/plugins/PATPhotonProducer.cc @@ -193,13 +193,8 @@ void pat::PATPhotonProducer::readIsolationLabels(const edm::ParameterSet& iConfi using namespace pat; PATPhotonProducer::PATPhotonProducer(const edm::ParameterSet& iConfig) - : - - ecalClusterToolsESGetTokens_{consumesCollector()}, - isolator_(iConfig.exists("userIsolation") ? iConfig.getParameter("userIsolation") - : edm::ParameterSet(), - consumesCollector(), - false), + : ecalClusterToolsESGetTokens_{consumesCollector()}, + isolator_(iConfig.getParameter("userIsolation"), consumesCollector(), false), useUserData_(iConfig.exists("userData")), ecalTopologyToken_{esConsumes()}, ecalGeometryToken_{esConsumes()} { @@ -221,14 +216,8 @@ PATPhotonProducer::PATPhotonProducer(const edm::ParameterSet& iConfig) addGenMatch_ = iConfig.getParameter("addGenMatch"); if (addGenMatch_) { embedGenMatch_ = iConfig.getParameter("embedGenMatch"); - if (iConfig.existsAs("genParticleMatch")) { - genMatchTokens_.push_back(consumes>( - iConfig.getParameter("genParticleMatch"))); - } else { - genMatchTokens_ = edm::vector_transform( - iConfig.getParameter>("genParticleMatch"), - [this](edm::InputTag const& tag) { return consumes>(tag); }); - } + genMatchTokens_.push_back(consumes>( + iConfig.getParameter("genParticleMatch"))); } // Efficiency configurables addEfficiencies_ = iConfig.getParameter("addEfficiencies"); diff --git a/PhysicsTools/PatAlgos/plugins/PATTauProducer.cc b/PhysicsTools/PatAlgos/plugins/PATTauProducer.cc index 6067a789f96f5..98732a5369664 100644 --- a/PhysicsTools/PatAlgos/plugins/PATTauProducer.cc +++ b/PhysicsTools/PatAlgos/plugins/PATTauProducer.cc @@ -141,10 +141,7 @@ namespace pat { using namespace pat; PATTauProducer::PATTauProducer(const edm::ParameterSet& iConfig) - : isolator_(iConfig.exists("userIsolation") ? iConfig.getParameter("userIsolation") - : edm::ParameterSet(), - consumesCollector(), - false), + : isolator_(iConfig.getParameter("userIsolation"), consumesCollector(), false), useUserData_(iConfig.exists("userData")), posAtECalEntranceComputer_(consumesCollector()) { firstOccurence_ = true; @@ -170,14 +167,8 @@ PATTauProducer::PATTauProducer(const edm::ParameterSet& iConfig) addGenMatch_ = iConfig.getParameter("addGenMatch"); if (addGenMatch_) { embedGenMatch_ = iConfig.getParameter("embedGenMatch"); - if (iConfig.existsAs("genParticleMatch")) { - genMatchTokens_.push_back(consumes>( - iConfig.getParameter("genParticleMatch"))); - } else { - genMatchTokens_ = edm::vector_transform( - iConfig.getParameter>("genParticleMatch"), - [this](edm::InputTag const& tag) { return consumes>(tag); }); - } + genMatchTokens_.push_back(consumes>( + iConfig.getParameter("genParticleMatch"))); } addGenJetMatch_ = iConfig.getParameter("addGenJetMatch"); if (addGenJetMatch_) {