Update runEddy script to work with new multi-threaded version.

The eddy_cpu command is called twice - once with --nthr=1, and again with --nthr=8

Note that these changes will not work with the openmp eddy build, so do not merge / use this MR until threddy has been released.

eddy/runEddy

@@ -10,12 +10,16 @@
 # output as the final line of output. Thne script returns
 # an exit code of non-0 if something goes wrong.
 #
+# The eddy_cpu executable is executed twice - once with
+# --nthr=1, and again with --nthr=8.
+#
 # Outputs of each eddy_<variant> executable is saved with
 # prefix ${outdir}/eddyOutput_<variant>. For example, the
 # output of eddy_cuda9.2 will be saved with prefix
 # ${outdir}/eddyOutput_cuda9.2, and the output of
-# eddy_openmp will be saved with prefix
-# ${outdir}/eddyOutput_openmp.
+# eddy_cpu --nthr=1 will be saved with prefix
+# ${outdir}/eddyOutput_cpu_nthr_1.
+#
 
 set -e
 
@@ -26,7 +30,7 @@ set -e
 # added by this script.
 exedir="$1"; shift;
 outdir="$1"; shift;
-eddy_args="$@"
+eddy_args=("$@")
 
 # Find all eddy_cuda* executables
 cuda_exes=""
@@ -37,13 +41,15 @@ do
   fi
 done
 
-# Find all eddy_cpu/eddy_openmp executables
+# Find all eddy_cpu executables. We run
+# eddy_cpu twice - here, we add
+# "eddy_cpu_nthr_X" to the list of
+# executables to run. The name is unpacked
+# in the loop below.
 cpu_exes=""
-if [ -x "${exedir}/eddy_openmp" ]; then
-  cpu_exes="${exedir}/eddy_openmp"
-fi
 if [ -x "${exedir}/eddy_cpu" ]; then
-  cpu_exes="${cpu_exes} ${exedir}/eddy_cpu"
+  cpu_exes="${cpu_exes} ${exedir}/eddy_cpu_nthr_1"
+  cpu_exes="${cpu_exes} ${exedir}/eddy_cpu_nthr_8"
 fi
 
 if [ "${cuda_exes}" == "" ] && [ "${cpu_exes}" == "" ]; then
@@ -57,15 +63,24 @@ fi
 cuda_jids=""
 cpu_jids=""
 submitted=""
-for exe in ${cuda_exes} ${cpu_exes};
+for exe_name in ${cuda_exes} ${cpu_exes};
 do
-  variant=`basename ${exe}`
+  variant=`basename ${exe_name}`
   variant=`echo ${variant} | sed 's/eddy_//'`
 
-  if [[ "${exe}" == *"cuda"* ]]; then
+  if [[ "${exe_name}" == *"cuda"* ]]; then
     fsl_sub="fsl_sub -l ${outdir} --coprocessor=cuda"
+    exe="${exe_name}"
   else
-    fsl_sub="fsl_sub -l ${outdir} -q long.q -s openmp,6"
+    # unpack eddy_cpu_nthr_N
+    # into eddy_cpu --nthr=N
+    fsl_sub="fsl_sub -l ${outdir} -q long.q"
+    exe="${exe_name%_nthr*}"
+    nthr="${variant#eddy_cpu_nthr_}"
+    eddy_args+=("--nthr=${nthr}")
+    if [ "${nthr}" != "1" ]; then
+      fsl_sub="${fsl_sub} -s openmp,${nthr}"
+    fi
   fi
 
   # fsl_sub will return an error (and not
@@ -74,11 +89,11 @@ do
   # or queue. So we allow it to fail.
   jid=$(${fsl_sub} ${exe} --out=${outdir}/eddyOutput_${variant} ${eddy_args} || true)
   if [ "${jid}" == "" ]; then
-    echo "Error submitting ${exe} - skipping"
+    echo "Error submitting ${exe_name} - skipping"
     continue
   fi
 
-  submitted="${submitted} ${exe}"
+  submitted="${submitted} ${exe_name}"
 
   if [[ "${exe}" == *"cuda"* ]]; then
     cuda_jids="${cuda_jids} ${jid}"