Changeset 36364

Timestamp:

Dec 10, 2013, 12:53:15 PM (13 years ago)

Author:

mhuber

Message:

misc ppStack issues -- if cfitsio fault then really want status as well as filename; psFree has to be used/uncomment back in in ppStack, actual cause of any possible abort must rather itself be fixed

Location:

trunk

Files:

• ppStack/src/ppStackLoop.c (modified) (2 diffs)
• ppStack/src/ppStackMatch.c (modified) (2 diffs)
• psLib/src/fits/psFits.c (modified) (1 diff)

trunk/ppStack/src/ppStackLoop.c

@@ -87 +87 @@
         pmCellFreeData(options->cells->data[i]);
     }
-    //    psFree(stack);
+    //MEH -- uncomment back out
+    //psFree(stack);
 
     // Pixel rejection
@@ -181 +182 @@
         return false;
     }
-    //    psFree(stack);
+    //MEH uncomment back out..
+    //psFree(stack);
     psLogMsg("ppStack", PS_LOG_INFO, "Stage 7: Cleanup, WCS & JPEGS: %f sec", psTimerClear("PPSTACK_STEPS"));
     ppStackMemDump("cleanup");

trunk/ppStack/src/ppStackMatch.c

@@ -6 +6 @@
 #define COVAR_FRAC 0.01                 // Truncation fraction for covariance matrix
 
-// #define TESTING                         // Enable debugging output
+// #define TESTING                      // Enable debugging output
 // #define TESTING_REUSE_CONV           // Enable debugging for re-using convolved outputs from previous run
-
 // #define TESTING_CZW 
 
@@ -156 +155 @@
     if (options->convolve) {
         pmReadout *conv = pmReadoutAlloc(NULL); // Conv readout, for holding results temporarily
+
+        // MEH -- earlier defn needed with new simple mode modifications to  use TESTING_REUSE_CONV...
+        //const char *typeStr = psMetadataLookupStr(NULL, ppsub, "KERNEL.TYPE"); // Kernel type
+        //pmSubtractionKernelsType type = pmSubtractionKernelsTypeFromString(typeStr); // Kernel type
+
 #ifdef TESTING_REUSE_CONV
         // This is a hack to use the temporary convolved images and kernel generated previously.

trunk/psLib/src/fits/psFits.c

@@ -243 +243 @@
         (&fptr, name, iomode, &status);
         if (fptr == NULL || status != 0) {
-            psFitsDumpErrors(PS_ERR_IO, _("Could not open file,'%s'"), name);
+            // MEH -- if cfitsio error and have status, really want to know it as well and should be added where lacking elsewhere
+            psFitsDumpErrors(PS_ERR_IO, _("Could not open file,'%s %d'"), name,status);
             return NULL;
         }