diff --git a/Configuration/PyReleaseValidation/python/relval_steps.py b/Configuration/PyReleaseValidation/python/relval_steps.py index 5dad1d85f40cf..539f03f72452d 100644 --- a/Configuration/PyReleaseValidation/python/relval_steps.py +++ b/Configuration/PyReleaseValidation/python/relval_steps.py @@ -1562,7 +1562,7 @@ def lhegensim2017(fragment,howMuch): steps['HYBRIDRepackHI2015VR']={'--eventcontent':'RAW', '--datatier':'RAW', - '--conditions':'auto:run2_data', + '--conditions':'auto:run2_hlt_hi', '--step':'RAW2DIGI,REPACK:DigiToHybridRawRepack', '--scenario':'HeavyIons', '--data':'', diff --git a/RecoLocalTracker/SiStripZeroSuppression/python/customiseHybrid.py b/RecoLocalTracker/SiStripZeroSuppression/python/customiseHybrid.py index f4416d6447efa..42efe243f56cc 100644 --- a/RecoLocalTracker/SiStripZeroSuppression/python/customiseHybrid.py +++ b/RecoLocalTracker/SiStripZeroSuppression/python/customiseHybrid.py @@ -4,6 +4,7 @@ def runOnHybridZS(process): process.load("RecoLocalTracker.SiStripZeroSuppression.SiStripZeroSuppression_cfi") process.load("RecoLocalTracker.SiStripClusterizer.SiStripClusterizer_cfi") + process.siStripZeroSuppression.Algorithms.APVInspectMode = "Hybrid" zsInputs = process.siStripZeroSuppression.RawDigiProducersList clusInputs = process.siStripClusters.DigiProducersList unpackedZS = cms.InputTag("siStripDigis", "ZeroSuppressed") @@ -33,7 +34,7 @@ def addHybridEmulationBeforeRepack(process): zs.Algorithms.APVInspectMode = "HybridEmulation" zs.Algorithms.APVRestoreMode = "" zs.Algorithms.CommonModeNoiseSubtractionMode = 'Median' - zs.Algorithms.MeanCM = 512 + zs.Algorithms.MeanCM = 0 zs.Algorithms.DeltaCMThreshold = 20 zs.Algorithms.Use10bitsTruncation = True zs.RawDigiProducersList = cms.VInputTag(cms.InputTag("siStripDigis", "VirginRaw")) diff --git a/RecoLocalTracker/SiStripZeroSuppression/src/SiStripAPVRestorer.cc b/RecoLocalTracker/SiStripZeroSuppression/src/SiStripAPVRestorer.cc index 0443d3634b135..24716ef61bd7d 100644 --- a/RecoLocalTracker/SiStripZeroSuppression/src/SiStripAPVRestorer.cc +++ b/RecoLocalTracker/SiStripZeroSuppression/src/SiStripAPVRestorer.cc @@ -46,8 +46,8 @@ SiStripAPVRestorer::SiStripAPVRestorer(const edm::ParameterSet& conf): size_window_(conf.getParameter("sizeWindow")), width_cluster_(conf.getParameter("widthCluster")) { - if ( restoreAlgo_ == "BaselineFollower" && inspectAlgo_ != "BaselineFollower" ) - throw cms::Exception("Incompatible Algorithm") << "The BaselineFollower restore method requires the BaselineFollower inspect method"; + if ( restoreAlgo_ == "BaselineFollower" && inspectAlgo_ != "BaselineFollower" && inspectAlgo_ != "Hybrid" ) + throw cms::Exception("Incompatible Algorithm") << "The BaselineFollower restore method requires the BaselineFollower (or Hybrid) inspect method"; } @@ -323,8 +323,8 @@ inline uint16_t SiStripAPVRestorer::hybridEmulationInspect(uint16_t firstAPV, co if ( useRealMeanCM_ && ( std::end(meanCMmap_) != itCMMap ) ) MeanAPVCM = itCMMap->second[iAPV]; - const float DeltaCM = median_[iAPV] - MeanAPVCM; - if ( ( DeltaCM < 0 ) && ( std::abs(DeltaCM) > deltaCMThreshold_ ) ) { + const float DeltaCM = median_[iAPV] - (MeanAPVCM+1024)/2; + if ( ( DeltaCM < 0 ) && ( std::abs(DeltaCM) > deltaCMThreshold_/2 ) ) { apvFlags_[iAPV] = "HybridEmulation"; ++nAPVflagged; }