Merge pull request #49019 from israr100/cfi_fix_csc

Provide fillDescriptions to CSC ESproducers for HLT

Author: cmsbuild

CalibMuon/CSCCalibration/interface/CSCChannelMapperESProducer.h

@@ -4,6 +4,7 @@
 #include <memory>
 
 #include "FWCore/Framework/interface/ESProducer.h"
+#include "FWCore/ParameterSet/interface/ConfigurationDescriptions.h"  // for fillDescriptions
 
 #include "CalibMuon/CSCCalibration/interface/CSCChannelMapperBase.h"
 #include "CalibMuon/CSCCalibration/interface/CSCChannelMapperRecord.h"
@@ -17,6 +18,8 @@ class CSCChannelMapperESProducer : public edm::ESProducer {
 
   BSP_TYPE produce(const CSCChannelMapperRecord &);
 
+  static void fillDescriptions(edm::ConfigurationDescriptions &descriptions);
+
 private:
   std::string algoName;
 };

CalibMuon/CSCCalibration/interface/CSCIndexerESProducer.h

@@ -4,6 +4,7 @@
 #include <memory>
 
 #include "FWCore/Framework/interface/ESProducer.h"
+#include "FWCore/ParameterSet/interface/ConfigurationDescriptions.h"  // needed for fillDescriptions
 
 #include "CalibMuon/CSCCalibration/interface/CSCIndexerBase.h"
 #include "CalibMuon/CSCCalibration/interface/CSCIndexerRecord.h"
@@ -17,6 +18,8 @@ class CSCIndexerESProducer : public edm::ESProducer {
 
   BSP_TYPE produce(const CSCIndexerRecord &);
 
+  static void fillDescriptions(edm::ConfigurationDescriptions &descriptions);
+
 private:
   std::string algoName;
 };

CalibMuon/CSCCalibration/plugins/CSCChannelMapperESProducer.cc

@@ -1,9 +1,8 @@
 #include "FWCore/Framework/interface/ModuleFactory.h"
-
 #include "CalibMuon/CSCCalibration/interface/CSCChannelMapperESProducer.h"
 #include "CalibMuon/CSCCalibration/interface/CSCChannelMapperFactory.h"
-
 #include "FWCore/MessageLogger/interface/MessageLogger.h"
+#include "FWCore/ParameterSet/interface/ParameterSetDescription.h"
 
 CSCChannelMapperESProducer::CSCChannelMapperESProducer(const edm::ParameterSet &pset) {
   algoName = pset.getParameter<std::string>("AlgoName");
@@ -21,5 +20,12 @@ CSCChannelMapperESProducer::BSP_TYPE CSCChannelMapperESProducer::produce(const C
   return CSCChannelMapperESProducer::BSP_TYPE(CSCChannelMapperFactory::get()->create(algoName));
 }
 
+// ---- add this ----
+void CSCChannelMapperESProducer::fillDescriptions(edm::ConfigurationDescriptions &descriptions) {
+  edm::ParameterSetDescription desc;
+  desc.add<std::string>("AlgoName", "CSCChannelMapperStartup");  // default
+  descriptions.addWithDefaultLabel(desc);
+}
+
 // define this as a plug-in
 DEFINE_FWK_EVENTSETUP_MODULE(CSCChannelMapperESProducer);

CalibMuon/CSCCalibration/plugins/CSCIndexerESProducer.cc

@@ -1,9 +1,8 @@
 #include "FWCore/Framework/interface/ModuleFactory.h"
-
 #include "CalibMuon/CSCCalibration/interface/CSCIndexerESProducer.h"
 #include "CalibMuon/CSCCalibration/interface/CSCIndexerFactory.h"
-
 #include "FWCore/MessageLogger/interface/MessageLogger.h"
+#include "FWCore/ParameterSet/interface/ParameterSetDescription.h"
 
 CSCIndexerESProducer::CSCIndexerESProducer(const edm::ParameterSet &pset) {
   algoName = pset.getParameter<std::string>("AlgoName");
@@ -21,5 +20,12 @@ CSCIndexerESProducer::BSP_TYPE CSCIndexerESProducer::produce(const CSCIndexerRec
   return CSCIndexerESProducer::BSP_TYPE(CSCIndexerFactory::get()->create(algoName));
 }
 
+// ---- add this ----
+void CSCIndexerESProducer::fillDescriptions(edm::ConfigurationDescriptions &descriptions) {
+  edm::ParameterSetDescription desc;
+  desc.add<std::string>("AlgoName", "CSCIndexerStartup");  // default
+  descriptions.addWithDefaultLabel(desc);
+}
+
 // define this as a plug-in
 DEFINE_FWK_EVENTSETUP_MODULE(CSCIndexerESProducer);