Changeset 24838 for trunk/ppImage/src/ppImageDetrendFringe.c

Timestamp:

Jul 17, 2009, 1:14:47 PM (17 years ago)

Author:

eugene

Message:

handle missing video cells; use modified Fringe I/O concept

File:

• trunk/ppImage/src/ppImageDetrendFringe.c (modified) (3 diffs)

trunk/ppImage/src/ppImageDetrendFringe.c

@@ -11 +11 @@
     PS_ASSERT_PTR_NON_NULL(options, false);
 
-    // Reference fringe measurements
-    psArray *references = psMemIncrRefCounter(psMetadataLookupPtr(NULL, fringe->analysis,
-                                                                  "FRINGE.MEASUREMENTS"));
+    // Reference fringe measurements (stored on the reference cell->analysis)
+    psArray *references = psMemIncrRefCounter(psMetadataLookupPtr(NULL, fringe->analysis, "FRINGE.MEASUREMENTS"));
     if (!references) {
-        references = pmFringesParse(fringe); // Reference fringes
-        if (!references) {
-            psError(PS_ERR_IO, false, "Unable to find fringe references.\n");
-            return false;
-        }
-        psMetadataAdd(fringe->analysis, PS_LIST_TAIL, "FRINGE.MEASUREMENTS", PS_DATA_UNKNOWN,
-                      "Fringe measurements", references);
+        psError(PS_ERR_IO, false, "Unable to find fringe references.\n");
+        return false;
     }
 
@@ -75 +69 @@
     }
 
-    psMetadataAdd(readout->parent->analysis, PS_LIST_TAIL, scienceFringes,
-                  PS_DATA_UNKNOWN | PS_META_REPLACE, "Fringe measurements", measurements);
+    psMetadataAdd(readout->parent->analysis, PS_LIST_TAIL, scienceFringes, PS_DATA_UNKNOWN | PS_META_REPLACE, "Fringe measurements", measurements);
     psFree(measurements);
     psFree(scienceFringes);
@@ -92 +85 @@
     psArray *fringes = psArrayAlloc(cells->n); // Fringes, to return
     for (int i = 0; i < cells->n; i++) {
+        fringes->data[i] = NULL;
+
         pmCell *cell = cells->data[i];  // Cell of interest
 
         // XXX for now, skip the video cells (cell->readouts->n > 1)
         if (cell->readouts->n > 1) {
-          psWarning ("Skipping Video Cell for ppImageDetrendFringe.c:getFringes");
-          continue;
+            psWarning ("Skipping Video Cell (%d) for ppImageDetrendFringe.c:getFringes", i);
+            continue;
         }