Changeset 34048

Timestamp:

Jun 20, 2012, 12:22:23 PM (14 years ago)

Author:

eugene

Message:

add validity tests for VIDEO check

File:

• branches/eam_branches/ipp-20120601/ppImage/src/ppImageDetrendReadout.c (modified) (1 diff)

branches/eam_branches/ipp-20120601/ppImage/src/ppImageDetrendReadout.c

@@ -35 +35 @@
       }
     }
+
     // Check to see if we're in a chip that contains video
+    // NOTE: this is an esoteric test for a GPC1 feature : this is irrelevant & will not work for other cameras
     bool hasVideo = false;
     {
+      // XXX test for GPC1? or CAN_HAVE_VIDEO in camera config?
+      if (!input) goto done_video_check;
+      if (!input->parent) goto done_video_check;
+      if (!input->parent->parent) goto done_video_check;
+      if (!input->parent->parent->hdu) goto done_video_check;
+      if (!input->parent->parent->hdu->header) goto done_video_check;
       if (psMetadataLookupStr(NULL,input->parent->parent->hdu->header,"CELLMODE")) {
-        psWarning("VIDEO: %d %d %d\n",(int) options->hasVideo,(int) options->useVideoDark, (int) options->useVideoMask);
+        psLogMsg ("ppImage.detrend", PS_LOG_DETAIL, "VIDEO: %d %d %d\n",(int) options->hasVideo,(int) options->useVideoDark, (int) options->useVideoMask);
         char *Vptr = strchr(psMetadataLookupStr(NULL,input->parent->parent->hdu->header,"CELLMODE"),'V');
         if (Vptr) {
           hasVideo = true;
-          psWarning("VIDEO: %d %d %d\n",(int) options->hasVideo,(int) options->useVideoDark, (int) options->useVideoMask);
+          psLogMsg ("ppImage.detrend", PS_LOG_INFO, "VIDEO: %d %d %d\n",(int) options->hasVideo,(int) options->useVideoDark, (int) options->useVideoMask);
         }
       }
     }
-
+    done_video_check:
     
     // Masking on the basis of pixel value needs to be done before anything else, so the values are pristine.