spm8Batch VBM bug

[mangstad]

UMBatchWarpVBM8.m contains a bug in this bit of code

If the OutputName is set to w, then the first loop never runs, so newImages2WriteUnique is an empty matrix, so then the second loop fails while trying to write log information.

The solution is probably to write the loop so that it enters anyway, but only perform the actual movefile command if OutputName is not w. This way newImages2WriteUnique is still built up and the logging call will not fail.

[dankessler]

If you could save me some reading...what does the w flag for OutputName signify?

[mangstad]

It specifies the prefix for the newly outputted files. The default is 'w', which is why I assume it skips the loop in the first place if it's set to w since it doesn't have to move the images at all. The default image with a w prefix are renamed to have the OutputName prefix instead of a w.

[rcwelsh]

Hi Mike,

I'll look fix this when I'm back to work on Monday. Thanks for finding the
bug.

-Robert

On 7/25/12 1:50 PM, "Mike Angstadt"
<reply+i-5835973-bf8d6137ca913a0b1d5442498da38ab18b945656-1485912@reply.git
hub.com> wrote:

UMBatchWarpVBM8.m contains a bug in [this bit of
code](https://github.com/UMPsychMethodsCore/MethodsCore/blob/spm8Batch_alph
a/spm8Batch/matlabScripts/UMBatchWarpVBM8.m#L239-267)

If the OutputName is set to w, then the first loop never runs, so
newImages2WriteUnique is an empty matrix, so then the second loop fails
while trying to write log information.

The solution is probably to write the loop so that it enters anyway, but
only perform the actual movefile command if OutputName is not w. This way
newImages2WriteUnique is still built up and the logging call will not fail.

---

Reply to this email directly or view it on GitHub:
#93

---

Electronic Mail is not secure, may not be read every day, and should not be used for urgent or sensitive issues