Merge pull request #17871 from fwyzard/set_respCorrM3_to_1.0

Set HCAL method 3 respons to 1.0 for data and MC.
cms-sw · Mar 15, 2017 · 869c7c5 · 869c7c5
2 parents e446eec + 5e37013
commit 869c7c5
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 4 deletions.
diff --git a/HLTrigger/Configuration/python/customizeHLTforMC.py b/HLTrigger/Configuration/python/customizeHLTforMC.py
@@ -28,6 +28,8 @@ def customizeHLTforMC(process):
   if 'hltHbhePhase1Reco' in process.__dict__:
     if process.hltHbhePhase1Reco._TypedParameterizable__type == 'HBHEPhase1Reconstructor':
       # 2017 "plan 1"
-      process.hltHbhePhase1Reco.algorithm.respCorrM3 = cms.double( 1.0 )
+      # assume retuning the pulse shapes will give the same response
+      # in data and MC for Method 2 and Method 3
+      pass
 
   return process
diff --git a/RecoLocalCalo/HcalRecProducers/python/HBHEMethod3Parameters_cfi.py b/RecoLocalCalo/HcalRecProducers/python/HBHEMethod3Parameters_cfi.py
@@ -6,5 +6,5 @@
     pedestalUpperLimit      = cms.double(2.7),
     timeSlewParsType        = cms.int32(3),     # 0: TestStand, 1:Data, 2:MC, 3:InputPars. Parametrization function is par0 + par1*log(fC+par2).
     timeSlewPars            = cms.vdouble(12.2999, -2.19142, 0, 12.2999, -2.19142, 0, 12.2999, -2.19142, 0), # HB par0, HB par1, HB par2, BE par0, BE par1, BE par2, HE par0, HE par1, HE par2
-    respCorrM3              = cms.double(0.95) # This factor is used to align the the Method3 with the Method2 response
+    respCorrM3              = cms.double(1.0)   # This factor is used to align the the Method3 with the Method2 response
 )
diff --git a/RecoLocalCalo/HcalRecProducers/src/HcalHitReconstructor.cc b/RecoLocalCalo/HcalRecProducers/src/HcalHitReconstructor.cc
@@ -295,7 +295,7 @@ void HcalHitReconstructor::fillDescriptions(edm::ConfigurationDescriptions& desc
   desc.add<double>("pedestalUpperLimit", 2.7); 
   desc.add<int>("timeSlewParsType",3);
   desc.add<std::vector<double>>("timeSlewPars", { 12.2999, -2.19142, 0, 12.2999, -2.19142, 0, 12.2999, -2.19142, 0 });
-  desc.add<double>("respCorrM3", 0.95);
+  desc.add<double>("respCorrM3", 1.0);
   descriptions.add("hltHbhereco",desc);
 }
 

diff --git a/RecoLocalCalo/HcalRecProducers/src/HcalSimpleReconstructor.cc b/RecoLocalCalo/HcalRecProducers/src/HcalSimpleReconstructor.cc
@@ -66,7 +66,7 @@ void HcalSimpleReconstructor::fillDescriptions(edm::ConfigurationDescriptions& d
   desc.add<double>("pedestalUpperLimit", 2.7); 
   desc.add<int>("timeSlewParsType",3);
   desc.add<std::vector<double>>("timeSlewPars", { 12.2999, -2.19142, 0, 12.2999, -2.19142, 0, 12.2999, -2.19142, 0 });
-  desc.add<double>("respCorrM3", 0.95);
+  desc.add<double>("respCorrM3", 1.0);
   descriptions.add("simpleHbhereco",desc);
 }