Changeset 31428 for branches/czw_branch/20110406/ippTools/src

Timestamp:

May 3, 2011, 2:25:17 PM (15 years ago)

Author:

watersc1

Message:

difftool: add the ability to specify both the input_warp_id and the template_warp_id. This allows us to explicitly say "subtract this one from that one."
stacktool: fix stackAssociation listing to supply the information, even if stackSumSkyfile isn't populated yet.
laptool: LAP now works cleanly from -definesequence through the final stack stage for runs that do not require a quickstack.

Location:

branches/czw_branch/20110406/ippTools/src

Files:

• difftool.c (modified) (1 diff)
• difftoolConfig.c (modified) (1 diff)
• laptool.c (modified) (4 diffs)

branches/czw_branch/20110406/ippTools/src/difftool.c

@@ -1625 +1625 @@
     PXOPT_COPY_S64(config->args, selectWhere, "-exp_id", "inputRawExp.exp_id", "==");
     PXOPT_COPY_S64(config->args, selectWhere, "-template_exp_id", "templateRawExp.exp_id", "==");
+    PXOPT_COPY_S64(config->args, selectWhere, "-template_warp_id", "templateWarpRun.warp_id", "==");
     PXOPT_COPY_STR(config->args, selectWhere, "-filter", "inputRawExp.filter", "==");
     PXOPT_COPY_STR(config->args, selectWhere, "-obs_mode", "inputRawExp.obs_mode", "==");

branches/czw_branch/20110406/ippTools/src/difftoolConfig.c

@@ -278 +278 @@
     psMetadataAddBool(definewarpwarpArgs, PS_LIST_TAIL, "-not-bothways",  0, "only do the single-direction subtraction?", false);
     psMetadataAddS64(definewarpwarpArgs, PS_LIST_TAIL,  "-template_exp_id",  0,  "search by template exposure ID", 0);
+    psMetadataAddS64(definewarpwarpArgs, PS_LIST_TAIL,  "-template_warp_id",  0,  "search by template warp ID", 0);
     psMetadataAddStr(definewarpwarpArgs, PS_LIST_TAIL, "-filter", 0, "search by filter", NULL);
     psMetadataAddF32(definewarpwarpArgs, PS_LIST_TAIL, "-distance", 0, "limit distance between input and template (deg)", NAN);

branches/czw_branch/20110406/ippTools/src/laptool.c

@@ -295 +295 @@
     return(false);
   }
-  psStringAppend(&query, " %s", whereClause);
+
+  if (whereClause) {
+    psStringSubstitute(&query,whereClause,"@WHERE@");
+  }
   psFree(where);
 
@@ -467 +470 @@
       }
       else {
-        return(false);
+        return(true); // We shouldn't really fail if we didn't change anything. Maybe there's nothing to change.
       }
     }
-    return(true);
-  }
-  else {
-    return(false);
-  }
+
+  }
+
+  return(true);
 }
 
@@ -496 +498 @@
   if (psListLength(where->list)) {
     psString whereClause = psDBGenerateWhereConditionSQL(where, NULL);
-    psStringAppend(&query, " WHERE %s", whereClause);
+    psStringAppend(&query, "\n AND %s", whereClause);
     psFree(whereClause);
   }
-
+  psStringAppend(&query, " ORDER BY rawExp.dateobs ");
+  
   if (limit) {
     psString limitString = psDBGenerateLimitSQL(limit);
-    psStringAppend(&query, " %s", limitString);
+    psStringAppend(&query, "\n %s", limitString);
     psFree(limitString);
   }
-  psStringAppend(&query, " ORDER BY rawExp.dateobs ");
+
   
   if (!p_psDBRunQuery(config->dbh, query)) {
@@ -615 +618 @@
 
   psMetadata *where = psMetadataAlloc();
-  PXOPT_COPY_S64(config->args, where, "-lap_id", "lap_id", "==");
+  PXOPT_COPY_S64(config->args, where, "-lap_id", "lapRun.lap_id", "==");
   
   psString query = pxDataGet("laptool_stacks.sql");