Changeset 32283 for trunk/ippTools

Timestamp:

Sep 1, 2011, 2:12:15 PM (15 years ago)

Author:

watersc1

Message:

Fix magictool to understand that we sometimes want to allow multiple diffs in the same label to be independently processed.

Location:

trunk/ippTools

Files:

• share/Makefile.am (modified) (1 diff)
• share/magictool_definebyquery_select.sql (modified) (1 diff)
• share/magictool_definebyquery_select_multidiff.sql (added)
• src/magictool.c (modified) (2 diffs)
• src/magictoolConfig.c (modified) (1 diff)

trunk/ippTools/share/Makefile.am

@@ -244 +244 @@
         magictool_definebyquery_insert.sql \
         magictool_definebyquery_select.sql \
+        magictool_definebyquery_select_multidiff.sql \
         magictool_inputs.sql \
         magictool_inputskyfile.sql \

trunk/ippTools/share/magictool_definebyquery_select.sql

@@ -49 +49 @@
     ) AS oldMagicRun
     ON oldMagicRun.exp_id = chipRun.exp_id
---    AND oldMagicRun.diff_id = diffWarps.diff_id
 -- WHERE hook %s
 GROUP BY chipRun.exp_id

trunk/ippTools/src/magictool.c

@@ -131 +131 @@
     PXOPT_LOOKUP_BOOL(simple, config->args, "-simple", false);
     PXOPT_LOOKUP_BOOL(pretend, config->args, "-pretend", false);
-
+    PXOPT_LOOKUP_BOOL(multidiff, config->args, "-multidiff", false);
     psMetadata *diffWhere = psMetadataAlloc(); // WHERE conditions for diffRuns
     PXOPT_COPY_STR(config->args, diffWhere, "-diff_label", "diffRun.label", "==");
@@ -142 +142 @@
     // Get list of exposures ready to magic
     {
-        psString query = pxDataGet("magictool_definebyquery_select.sql");
+        psString query;
+        if (multidiff) {
+          query = pxDataGet("magictool_definebyquery_select_multidiff.sql");
+        }
+        else {
+          query = pxDataGet("magictool_definebyquery_select.sql");
+        }
         if (!query) {
             psError(PXTOOLS_ERR_SYS, false, "failed to retreive SQL statement");

trunk/ippTools/src/magictoolConfig.c

@@ -61 +61 @@
     psMetadataAddBool(definebyqueryArgs, PS_LIST_TAIL, "-available", 0, "process what's immediately available?", false);
     psMetadataAddBool(definebyqueryArgs, PS_LIST_TAIL, "-rerun", 0, "generate new run even if existing?", false);
+    psMetadataAddBool(definebyqueryArgs, PS_LIST_TAIL, "-multidiff", 0, "allow multiple diffs to be magicked with the same label", false);
     psMetadataAddBool(definebyqueryArgs, PS_LIST_TAIL, "-simple", 0, "use the simple output format", false);
     psMetadataAddBool(definebyqueryArgs, PS_LIST_TAIL, "-pretend", 0, "list results but do not queue", false);