Reduce verbosity and throttle alarms

davidrohr · shahor02 · commit d523c0625bda · 2022-05-28T04:30:40.000+04:00
diff --git a/Detectors/CPV/reconstruction/src/FullCluster.cxx b/Detectors/CPV/reconstruction/src/FullCluster.cxx
@@ -173,7 +173,10 @@ char FullCluster::getNumberOfLocalMax(gsl::span<int> maxAt) const
       maxAt[iDigitN] = i;
       iDigitN++;
       if (iDigitN >= maxAt.size()) { // Note that size of output arrays is limited:
-        LOG(error) << "Too many local maxima, cluster multiplicity " << mMulDigit;
+        static int nAlarms = 0;
+        if (nAlarms++ < 5) {
+          LOG(alarm) << "Too many local maxima, cluster multiplicity " << mMulDigit;
+        }
         return 0;
       }
     }
diff --git a/Detectors/MUON/MCH/Workflow/src/PreClusterFinderSpec.cxx b/Detectors/MUON/MCH/Workflow/src/PreClusterFinderSpec.cxx
@@ -173,7 +173,10 @@ class PreClusterFinderTask
           break;
         case CHECK_NO_LEFTOVER_DIGITS_ERROR:
           if (digitsSizesDiffer) {
-            LOG(error) << "some digits have been lost during the preclustering";
+            static int nAlarms = 0;
+            if (nAlarms++ < 5) {
+              LOG(alarm) << "some digits have been lost during the preclustering";
+            }
           }
           break;
         case CHECK_NO_LEFTOVER_DIGITS_FATAL:
diff --git a/Detectors/PHOS/reconstruction/src/Clusterer.cxx b/Detectors/PHOS/reconstruction/src/Clusterer.cxx
@@ -677,7 +677,10 @@ char Clusterer::getNumberOfLocalMax(Cluster& clu, std::vector<CluElement>& cluel
       mMaxAt[iDigitN] = i + iFirst;
       iDigitN++;
       if (iDigitN >= NLOCMAX) { // Note that size of output arrays is limited:
-        LOG(error) << "Too many local maxima, cluster multiplicity " << mIsLocalMax.size();
+        static int nAlarms = 0;
+        if (nAlarms++ < 5) {
+          LOG(alarm) << "Too many local maxima, cluster multiplicity " << mIsLocalMax.size();
+        }
         return -2;
       }
     }

Original file line number	Diff line number	Diff line change
`@@ -173,7 +173,10 @@ char FullCluster::getNumberOfLocalMax(gsl::span<int> maxAt) const`
`173`	`173`	`maxAt[iDigitN] = i;`
`174`	`174`	`iDigitN++;`
`175`	`175`	`if (iDigitN >= maxAt.size()) { // Note that size of output arrays is limited:`
`176`		`- LOG(error) << "Too many local maxima, cluster multiplicity " << mMulDigit;`
	`176`	`+ static int nAlarms = 0;`
	`177`	`+ if (nAlarms++ < 5) {`
	`178`	`+ LOG(alarm) << "Too many local maxima, cluster multiplicity " << mMulDigit;`
	`179`	`+ }`
`177`	`180`	`return 0;`
`178`	`181`	`}`
`179`	`182`	`}`
Original file line number	Diff line number	Diff line change
`@@ -677,7 +677,10 @@ char Clusterer::getNumberOfLocalMax(Cluster& clu, std::vector<CluElement>& cluel`
`677`	`677`	`mMaxAt[iDigitN] = i + iFirst;`
`678`	`678`	`iDigitN++;`
`679`	`679`	`if (iDigitN >= NLOCMAX) { // Note that size of output arrays is limited:`
`680`		`- LOG(error) << "Too many local maxima, cluster multiplicity " << mIsLocalMax.size();`
	`680`	`+ static int nAlarms = 0;`
	`681`	`+ if (nAlarms++ < 5) {`
	`682`	`+ LOG(alarm) << "Too many local maxima, cluster multiplicity " << mIsLocalMax.size();`
	`683`	`+ }`
`681`	`684`	`return -2;`
`682`	`685`	`}`
`683`	`686`	`}`