Changeset 35499

Timestamp:

May 3, 2013, 10:24:07 AM (13 years ago)

Author:

bills

Message:

Fix bugs in extracting sources from postage stamp ROI
Stop failing dependency checks with PSTAMP_GONE for warps
with tess_id = RINGS.V0

Location:

tags/ipp-20130307/pstamp

Files:

• . (modified) (1 prop)
• scripts/pstamp_checkdependent.pl (modified) (2 diffs)
• scripts/pstamp_job_run.pl (modified) (1 diff)
• src/ppstampMakeStamp.c (modified) (2 diffs)
• src/ppstampParseCamera.c (modified) (1 diff)

tags/ipp-20130307/pstamp/scripts/pstamp_checkdependent.pl

@@ -344 +344 @@
 }
 
+if (0) {
     my $tess_id = $metadata->{tess_id};
     if ($tess_id eq 'RINGS.V0') {
@@ -349 +350 @@
         return $PSTAMP_GONE
     }
+}
     if (($state =~ /error/) or ($state =~ /purged/) or ($state =~ /scrubbed/) or ($state eq 'drop') or
          ($data_state =~ /error/) or ($data_state =~ /purged/) or ($data_state =~ /scrubbed/) or ($data_state eq 'drop')) {

tags/ipp-20130307/pstamp/scripts/pstamp_job_run.pl

@@ -151 +151 @@
         } elsif ($params->{cmf}) {
             $argString .= " -write_cmf";
+            $fileArgs  .= " -sources $params->{cmf}";
             push @file_list, $params->{cmf};
         } else {

tags/ipp-20130307/pstamp/src/ppstampMakeStamp.c

@@ -359 +359 @@
 
         if (options->writeCMF) {
-            pmReadout *astromReadout = NULL;
+            pmReadout *sourcesReadout = NULL;
             if (astrom->fpa != input->fpa) {
-                astromReadout = pmFPAviewThisReadout(srcView, astrom->fpa);
-            }
-            if (!copySources(outReadout, astromReadout, astromReadout, extractRegion)) {
+                sourcesReadout = pmFPAviewThisReadout(srcView, astrom->fpa);
+            } else {
+                sourcesReadout = readout;
+            }
+            if (!copySources(outReadout, sourcesReadout, sourcesReadout, extractRegion)) {
                 psError(PS_ERR_UNKNOWN, false, "failed to extract sources from region of interest.\n");
                 status = false;
@@ -784 +786 @@
 
     // first look for detections in the input readout. Those would have come from the -sources file
+    if (!inReadout->analysis) {
+        psLogMsg("ppstampMakeStamp", PS_LOG_WARN, "no analysis metadata found on input\n");
+        return false;
+    }
     pmDetections *inDetections = psMetadataLookupPtr (&status, inReadout->analysis, "PSPHOT.DETECTIONS");
     if (!inDetections) {

tags/ipp-20130307/pstamp/src/ppstampParseCamera.c

@@ -119 +119 @@
         // see if -sources file was supplied. 
         // If so define the file.
-        psPtr sourcesFile = psMetadataLookupStr(&status, config->arguments, "SOURCES");
+        psPtr sourcesFile = psMetadataLookupPtr(&status, config->arguments, "SOURCES");
         if (sourcesFile) {
             pmFPAfile *sources = pmFPAfileBindFromArgs(&status, input, config, "PPSTAMP.INPUT.SOURCES", "SOURCES");