Merge pull request #36968 from mmusich/silenceNoClusters

SiPixelClusterizer: No digis to clusterize warning into error
cms-sw · Mar 2, 2022 · 6310cdb · 6310cdb
2 parents 21106cf + 5bc0563
commit 6310cdb
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 6 deletions.
diff --git a/RecoLocalTracker/SiPixelClusterizer/plugins/PixelThresholdClusterizer.cc b/RecoLocalTracker/SiPixelClusterizer/plugins/PixelThresholdClusterizer.cc
@@ -130,9 +130,11 @@ void PixelThresholdClusterizer::clusterizeDetUnitT(const T& input,
   typename T::const_iterator begin = input.begin();
   typename T::const_iterator end = input.end();
 
-  // Do not bother for empty detectors
-  if (begin == end)
-    edm::LogWarning("clusterizeDetUnit()") << "No digis to clusterize";
+  // this should never happen and the raw2digi does not create empty detsets
+  if (begin == end) {
+    edm::LogError("PixelThresholdClusterizer") << "@SUB=PixelThresholdClusterizer::clusterizeDetUnitT()"
+                                               << " No digis to clusterize";
+  }
 
   //  Set up the clusterization on this DetId.
   if (!setup(pixDet))

diff --git a/RecoLocalTracker/SiPixelClusterizer/plugins/PixelThresholdClusterizerForBricked.cc b/RecoLocalTracker/SiPixelClusterizer/plugins/PixelThresholdClusterizerForBricked.cc
@@ -49,9 +49,12 @@ void PixelThresholdClusterizerForBricked::clusterizeDetUnitT(const T& input,
 
   edm::LogInfo("PixelThresholdClusterizerForBricked::clusterizeDetUnitT()");
 
-  // Do not bother for empty detectors
-  if (begin == end)
-    edm::LogWarning("clusterizeDetUnit()") << "No digis to clusterize";
+  // this should never happen and the raw2digi does not create empty detsets
+  if (begin == end) {
+    edm::LogError("PixelThresholdClusterizerForBricked")
+        << "@SUB=PixelThresholdClusterizerForBricked::clusterizeDetUnitT()"
+        << " No digis to clusterize";
+  }
 
   //  Set up the clusterization on this DetId.
   if (!setup(pixDet))