Changeset 40810 for trunk/psModules

Timestamp:

Jun 20, 2019, 4:15:45 PM (7 years ago)

Author:

eugene

Message:

do not raise an error if psSourceRoughStats does not have enough data to measure the min/max; just print the failure

File:

• trunk/psModules/src/objects/pmSource.c (modified) (2 diffs)

trunk/psModules/src/objects/pmSource.c

@@ -783 +783 @@
 
         if (!psVectorStats (stats, starsn_moments, NULL, NULL, 0)) {
-            psError(PS_ERR_UNKNOWN, false, "failed to measure SN / moments stats");
-            psFree (stats);
-            psFree (starsn_peaks);
-            return false;
+          psErrorClear(); // XXX it is probably excessive for psVectorStats to raise an error here (no valid values)
+          fprintf (stderr, "failed to measure SN / moments stats (no valid moments)\n");
+          // psError(PS_ERR_UNKNOWN, false, "failed to measure SN / moments stats");
+          // psFree (stats);
+          // psFree (starsn_peaks);
+          // return false;
         }
         psLogMsg ("pmObjects", 3, "SN range (moments): %f - %f\n", stats->min, stats->max);
@@ -797 +799 @@
         stats = psStatsAlloc (PS_STAT_MIN | PS_STAT_MAX);
         if (!psVectorStats (stats, starsn_peaks, NULL, NULL, 0)) {
-            psError(PS_ERR_UNKNOWN, false, "failed to measure SN / moments stats");
-            psFree (stats);
-            psFree (starsn_peaks);
-            return false;
-        }
-        psLogMsg ("psModules.objects", 3, "SN range (peaks)  : %f - %f (%ld)\n",
-                  stats->min, stats->max, starsn_peaks->n);
+          psErrorClear(); // XXX it is probably excessive for psVectorStats to raise an error here (no valid values)
+          fprintf (stderr, "failed to measure SN / peak stats (no valid peaks)\n");
+          // psError(PS_ERR_UNKNOWN, false, "failed to measure SN / moments stats");
+          // psFree (stats);
+          // psFree (starsn_peaks);
+          // return false;
+        }
+        psLogMsg ("psModules.objects", 3, "SN range (peaks)  : %f - %f (%ld)\n", stats->min, stats->max, starsn_peaks->n);
         psFree (stats);
     }