spm_BIDS.m DWI allows 'dir'

[CPernet]

in the case of separate additional acquisitions AP/PA it currently fails, e.g.

f =
{'/sub-048/dwi/sub-048_dir-AP_dwi.nii'}
{'/sub-048/dwi/sub-048_dir-PA_dwi.nii'}
{'/sub-048/dwi/sub-048_dwi.nii'       }

simply adding 'dir' does the job

[gllmflndn]

Hello @CPernet 👋

There is a similar PR on the development version of SPM: spm/spm#9

Would it be OK to merge it there?

[CPernet]

looking at the whole PR, all those are indeed BIDS keys you could just merge

[CPernet]

Anyone took over the repo?

[pzeidman]

Thank you Cyril for following up on this PR! Unfortunately we've had a hold-on up on being able to accept PRs, but I'll take a look at this and see if we can get it patched ASAP - I'll be in touch next week.

Best
Peter @ SPM

[CPernet]

Should not we try to crowd source? Like some other project so it less depending on UCL people - I'm in Seoul right now if anyone want to have that conversation