Set HCAL method 3 respons to 1.0 for data and MC.

Assume that retuning the pulse shapes will give the same response for Method 2 and Method 3, both for data and MC.
cms-sw · Mar 9, 2017 · 888ba46 · 888ba46
1 parent 8fe3aa1
commit 888ba46
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 5 deletions.
diff --git a/HLTrigger/Configuration/python/customizeHLTforHCALPhaseI.py b/HLTrigger/Configuration/python/customizeHLTforHCALPhaseI.py
@@ -88,7 +88,7 @@ def customizeHLTforHEforPhaseI(process):
             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
         ),
 
         # Reconstruction algorithm configuration data to fetch from DB, if any

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);
 }