Changeset 40848

Timestamp:

Jul 26, 2019, 12:11:22 PM (7 years ago)

Author:

ippitc

Message:

merge change from trunk (do not fail on fully-masked chip)

File:

• tags/ipp-20170121/psphot/src/psphotReadoutCleanup.c (modified) (3 diffs, 1 prop)

tags/ipp-20170121/psphot/src/psphotReadoutCleanup.c

@@ -55 +55 @@
     // XXX where do we free these, in here (psMetadataRemove?)
 
+    // XXX this is currently only set by psphotModelBackground / psphotModelBackgroundReadoutFileIndex / psphotModelBackgroundReadout
+    // if the image background cannot be measured (no valid pixels)
     int quality = psMetadataLookupS32 (&status, readout->analysis, "PSPHOT_QUALITY");
     if (quality) {
@@ -74 +76 @@
             return false;
         }
+    }
+
+    // Check to see if any sources were detected
+    // This is not necessarily a quality error: e.g., ppSub
+    if (0 && !psf && !sources) {
+      psError(PSPHOT_ERR_DATA, false, "Unable to detect sources in the image");
+      return false;
     }
 
@@ -115 +124 @@
 
     psFree (header);
+    if (quality) return false;
+
     return true;
 }