output for DTI vectors changed (dcm2nii -> dcm2niix) #1463
Conversation
New: Saving 65 DTI gradients. Please validate if you are conducting DTI analyses.
| @@ -292,7 +292,7 @@ def _parse_stdout(self, stdout): | |||
| bvals.append(out_file + ".bval") | |||
| find_b = False | |||
| # next scan will have bvals/bvecs | |||
| elif 'DTI gradient directions' in line: | |||
| elif 'DTI gradients' in line: | |||
There was a problem hiding this comment.
would suggest changing to elif 'DTI gradient' in line: to make compatible with former output as well
There was a problem hiding this comment.
Oh,
that's nasty. I thought 'DTI gradient directions' was the old dcm2nii
syntax and copy&paste introduced this error, but now I realize that this
is "old" dcm2niiX output.
+1 for another elif statement
In nipype/interfaces/dcm2nii.py
#1463 (comment):@@ -292,7 +292,7 @@ def _parse_stdout(self, stdout):
bvals.append(out_file + ".bval")
find_b = False
# next scan will have bvals/bvecs
elif 'DTI gradient directions' in line:elif 'DTI gradients' in line:would suggest changing to |elif 'DTI gradient' in line:| to make
compatible with former output as well
|
i fixed this in the upstream master. let's consider this a beta interface since @neurosclab is changing things depending on user needs. |
New: Saving 65 DTI gradients. Please validate if you are conducting DTI
analyses.
see #1389