Merge pull request #584 from lsst/tickets/DM-29069

Replace assert with exception for invalid argument
lsst · May 7, 2021 · fc6d2d7 · fc6d2d7
2 parents 2bc2b1e + 7befe53
commit fc6d2d7
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/include/lsst/afw/math/Statistics.h b/include/lsst/afw/math/Statistics.h
@@ -140,11 +140,17 @@ class StatisticsControl {
     bool getCalcErrorFromInputVariance() const noexcept { return _calcErrorFromInputVariance; }
 
     void setNumSigmaClip(double numSigmaClip) {
-        assert(numSigmaClip > 0);
+        if (!(numSigmaClip > 0)) {
+            throw LSST_EXCEPT(pex::exceptions::InvalidParameterError,
+                              "numSigmaClip has to be positive.");
+        }
         _numSigmaClip = numSigmaClip;
     }
     void setNumIter(int numIter) {
-        assert(numIter > 0);
+        if (!(numIter > 0)) {
+            throw LSST_EXCEPT(pex::exceptions::InvalidParameterError,
+                              "numIter has to be positive.");
+        }
         _numIter = numIter;
     }
     void setAndMask(int andMask) { _andMask = andMask; }