Skip to content

MRtrix 3.0 dwi2fod single output error #2822

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
jguillon opened this issue Dec 13, 2018 · 2 comments · Fixed by #3176
Closed

MRtrix 3.0 dwi2fod single output error #2822

jguillon opened this issue Dec 13, 2018 · 2 comments · Fixed by #3176
Milestone

Comments

@jguillon
Copy link
Contributor

Hi, I got this error when executing mrtrix3.EstimateFOD interface with the latest MRtrix version (3.0_RC2-117-gf098f097) installed on macOS Mojave 10.14.2 through a brew install mrtrix3:

dwi2fod: [ERROR] CSD algorithm expects a single input response function and single output FOD image

It seems that MRtrix now expects "pairs of input tissue response and output ODF images" according to the documentation. So I guess that the following lines need to be updated:

wm_txt = File(
argstr='%s', position=-6, mandatory=True, desc='WM response text file')
wm_odf = File(
'wm.mif',
argstr='%s',
position=-5,
usedefault=True,
mandatory=True,
desc='output WM ODF')
gm_txt = File(argstr='%s', position=-4, desc='GM response text file')
gm_odf = File('gm.mif', usedefault=True, argstr='%s',
position=-3, desc='output GM ODF')
csf_txt = File(argstr='%s', position=-2, desc='CSF response text file')
csf_odf = File('csf.mif', usedefault=True, argstr='%s',
position=-1, desc='output CSF ODF')

@matteomancini
Copy link
Contributor

The issue is that dwi2fod needs to be able to handle both CSD (where the only output is the white matter FOD) and multi-tissue CSD (where there is one file for each tissue). To solve the issue, gm_odf and csf_odf need to have the useDefault field equal to False. I have tested the solution and it works, I already fixed the code in my own fork of Nipype, if it is ok I can make a pull request. A related issue that I fixed there is the useDefault=True for the max_sh argument for mrtrix3.EstimateFOD, which causes troubles for multi-shell data. I already run the make check-before-commit and the code I worked on is fine.

@effigies
Copy link
Member

Hi @jguillon, sorry for how long this took. The 1.5.0 release will have a ConstrainedSphericalDeconvolution interface that should work correctly. In case you didn't see other discussion, the concern with fixing EstimateFOD is that people using msmt_csd may have been depending on the bug in their workflows.

The new interface is identical, but without the bug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants