Tim Coalson

This looks like an added sanity check was written incorrectly. You could work around it by using 1.4.2 for this command, or by using the -volume argument multiple times in...

You may have already noticed this, but there are also -volume-all options to -cifti-separate and similar, if your processing can use the volume labels to do whatever structure separation you...

Note that building without OSMesa will disable some features that can be important. You might want to install all the dependencies that ubuntu does when they build it, and maybe...

This also looks wrong: `-D WORKBENCH_MESA_DIR=/usr/include/GL/osmesa.h` This should point to a directory, and should probably just be /usr, as in the readme.

I believe `QT_LOGGING_RULES` needs to be an environment variable for it to suppress the logging of these messages (not part of the build process): ``` $ export QT_LOGGING_RULES='*.debug=false;qt.qpa.*=false' $ wb_view...

Thanks for bringing this up. The permissions should be improved in the next release.

Personally, I would use `case` instead of `if`/`else`, and there are unnecessary semicolons at the end of the math lines. Nifti also defines hertz, ppm, and rads as additional units...

Using this construct instead might resolve this issue, but would need to be tested (and if there are any blank lines, that may throw it off): ``` NumContrasts=`echo "$ContrastNames" |...

Also, the reason that just adding quotes around ContrastNames should work is that bash converts newlines to spaces when expanding a variable without quoting, and even a simple `var=$(ls)` will...

To me, that reads only as "here is a way to work around it", while not saying anything about whether or not they are inclined to make it behave as...