diff --git a/HLTrigger/Configuration/python/customizeHLTforCMSSW.py b/HLTrigger/Configuration/python/customizeHLTforCMSSW.py index 1cb644d600e12..b3c0b20700909 100644 --- a/HLTrigger/Configuration/python/customizeHLTforCMSSW.py +++ b/HLTrigger/Configuration/python/customizeHLTforCMSSW.py @@ -118,6 +118,10 @@ def customisePixelGainForRun2Input(process): producer.VCaltoElectronOffset_L1 = -670 for producer in producers_by_type(process, "SiPixelRawToClusterCUDA"): + producer.VCaltoElectronGain = cms.double(1.) + producer.VCaltoElectronGain_L1 = cms.double(1.) + producer.VCaltoElectronOffset = cms.double(0.) + producer.VCaltoElectronOffset_L1 = cms.double(0.) producer.isRun2 = True return process @@ -236,6 +240,34 @@ def customizeHLTfor41815(process): return process +def customizeHLTfor41632(process): + for producerType in [ + 'SiPixelRawToClusterCUDA', + 'SiPixelRawToClusterCUDAPhase1', + 'SiPixelRawToClusterCUDAHIonPhase1', + ]: + for producer in producers_by_type(process, producerType): + # use explicit cms.double as parameters may not already be present, and + # set values to the correct Run-3 values (even when the parameters are already defined) + producer.VCaltoElectronGain = cms.double(1.) + producer.VCaltoElectronGain_L1 = cms.double(1.) + producer.VCaltoElectronOffset = cms.double(0.) + producer.VCaltoElectronOffset_L1 = cms.double(0.) + + return process + +def customizeHLTfor42410(process): + for producerType in [ + 'SiPixelRawToClusterCUDA', + 'SiPixelRawToClusterCUDAPhase1', + 'SiPixelRawToClusterCUDAHIonPhase1', + ]: + for producer in producers_by_type(process, producerType): + if hasattr(producer, 'isRun2'): + del producer.isRun2 + + return process + # CMSSW version specific customizations def customizeHLTforCMSSW(process, menuType="GRun"): @@ -247,5 +279,7 @@ def customizeHLTforCMSSW(process, menuType="GRun"): process = customizeHLTfor41058(process) process = customizeHLTfor41495(process) process = customizeHLTfor41815(process) + process = customizeHLTfor41632(process) + process = customizeHLTfor42410(process) return process diff --git a/RecoLocalTracker/SiPixelClusterizer/plugins/SiPixelRawToClusterCUDA.cc b/RecoLocalTracker/SiPixelClusterizer/plugins/SiPixelRawToClusterCUDA.cc index 8aa39335d27d3..89e9c3f490bb4 100644 --- a/RecoLocalTracker/SiPixelClusterizer/plugins/SiPixelRawToClusterCUDA.cc +++ b/RecoLocalTracker/SiPixelClusterizer/plugins/SiPixelRawToClusterCUDA.cc @@ -78,7 +78,6 @@ class SiPixelRawToClusterCUDAT : public edm::stream::EDProducer::SiPixelRawToClusterCUDAT(const edm::Par gainsToken_(esConsumes()), cablingMapToken_(esConsumes( edm::ESInputTag("", iConfig.getParameter("CablingMapLabel")))), - isRun2_(iConfig.getParameter("isRun2")), includeErrors_(iConfig.getParameter("IncludeErrors")), useQuality_(iConfig.getParameter("UseQualityInfo")), clusterThresholds_{iConfig.getParameter("clusterThreshold_layer1"), @@ -116,7 +114,6 @@ SiPixelRawToClusterCUDAT::SiPixelRawToClusterCUDAT(const edm::Par template void SiPixelRawToClusterCUDAT::fillDescriptions(edm::ConfigurationDescriptions& descriptions) { edm::ParameterSetDescription desc; - desc.add("isRun2", true); desc.add("IncludeErrors", true); desc.add("UseQualityInfo", false); // Note: this parameter is obsolete: it is ignored and will have no effect. @@ -269,8 +266,7 @@ void SiPixelRawToClusterCUDAT::acquire(const edm::Event& iEvent, wordFedAppender.initializeWordFed(fedIds_[i], index[i], start[i], words[i]); } - gpuAlgo_.makePhase1ClustersAsync(isRun2_, - clusterThresholds_, + gpuAlgo_.makePhase1ClustersAsync(clusterThresholds_, gpuMap, gpuModulesToUnpack, gpuGains, diff --git a/RecoLocalTracker/SiPixelClusterizer/plugins/SiPixelRawToClusterGPUKernel.cu b/RecoLocalTracker/SiPixelClusterizer/plugins/SiPixelRawToClusterGPUKernel.cu index 419cc1109a732..5d7801a10bf72 100644 --- a/RecoLocalTracker/SiPixelClusterizer/plugins/SiPixelRawToClusterGPUKernel.cu +++ b/RecoLocalTracker/SiPixelClusterizer/plugins/SiPixelRawToClusterGPUKernel.cu @@ -527,7 +527,6 @@ namespace pixelgpudetails { // Interface to outside template void SiPixelRawToClusterGPUKernel::makePhase1ClustersAsync( - bool isRun2, const SiPixelClusterThresholds clusterThresholds, const SiPixelROCsStatusAndMapping *cablingMap, const unsigned char *modToUnp, diff --git a/RecoLocalTracker/SiPixelClusterizer/plugins/SiPixelRawToClusterGPUKernel.h b/RecoLocalTracker/SiPixelClusterizer/plugins/SiPixelRawToClusterGPUKernel.h index 98d079816ddd3..802ad2eb42c7e 100644 --- a/RecoLocalTracker/SiPixelClusterizer/plugins/SiPixelRawToClusterGPUKernel.h +++ b/RecoLocalTracker/SiPixelClusterizer/plugins/SiPixelRawToClusterGPUKernel.h @@ -104,8 +104,7 @@ namespace pixelgpudetails { SiPixelRawToClusterGPUKernel& operator=(const SiPixelRawToClusterGPUKernel&) = delete; SiPixelRawToClusterGPUKernel& operator=(SiPixelRawToClusterGPUKernel&&) = delete; - void makePhase1ClustersAsync(bool isRun2, - const SiPixelClusterThresholds clusterThresholds, + void makePhase1ClustersAsync(const SiPixelClusterThresholds clusterThresholds, const SiPixelROCsStatusAndMapping* cablingMap, const unsigned char* modToUnp, const SiPixelGainForHLTonGPU* gains, diff --git a/RecoLocalTracker/SiPixelClusterizer/python/siPixelClustersPreSplitting_cff.py b/RecoLocalTracker/SiPixelClusterizer/python/siPixelClustersPreSplitting_cff.py index 263e0f77e5fbf..4460dd6ab0240 100644 --- a/RecoLocalTracker/SiPixelClusterizer/python/siPixelClustersPreSplitting_cff.py +++ b/RecoLocalTracker/SiPixelClusterizer/python/siPixelClustersPreSplitting_cff.py @@ -28,7 +28,6 @@ run3_common.toModify(siPixelClustersPreSplittingCUDA, # use the pixel channel calibrations scheme for Run 3 - isRun2 = False, clusterThreshold_layer1 = 4000, VCaltoElectronGain = 1, # all gains=1, pedestals=0 VCaltoElectronGain_L1 = 1,