Skip to content

Commit 2900b12

Browse files
committed
update variable names
1 parent 5662935 commit 2900b12

File tree

2 files changed

+42
-47
lines changed

2 files changed

+42
-47
lines changed

nipype/interfaces/fsl/epi.py

Lines changed: 17 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -630,8 +630,6 @@ class EddyInputSpec(FSLCommandInputSpec):
630630
desc="Do NOT perform a post-eddy alignment of "
631631
"shells")
632632
fwhm = traits.Float(
633-
default_value=0.0,
634-
usedefault=True,
635633
desc=('FWHM for conditioning filter when estimating '
636634
'the parameters'),
637635
argstr='--fwhm=%s')
@@ -688,32 +686,32 @@ class EddyInputSpec(FSLCommandInputSpec):
688686
argstr='--mb_offs=%d',
689687
desc=('Multi-band offset (-1 if bottom slice removed, 1 if '
690688
'top slice removed'),
691-
requires=['mb'],
689+
requires=['multiband_factor'],
692690
min_ver='5.0.10')
693691

694692
mporder = traits.Int(
695693
argstr='--mporder=%s',
696694
desc='Order of slice-to-vol movement model',
697695
requires=['use_cuda'],
698696
min_ver='5.0.11')
699-
slice2vol_iterations = traits.Int(
700-
argstr='--s2v_niter=%s',
697+
slice2vol_niter = traits.Int(
698+
argstr='--s2v_niter=%d',
701699
desc='Number of iterations for slice-to-vol',
702700
requires=['mporder'],
703701
min_ver='5.0.11')
704-
s2v_lambda = traits.Int(
705-
agstr='--s2v_lambda',
702+
slice2vol_lambda = traits.Int(
703+
argstr='--s2v_lambda=%d',
706704
desc='Regularisation weight for slice-to-vol movement (reasonable range 1-10)',
707705
requires=['mporder'],
708706
min_ver='5.0.11')
709-
s2v_interp = traits.Enum(
707+
slice2vol_interp = traits.Enum(
710708
'trilinear',
711709
'spline',
712710
argstr='--s2v_interp=%s',
713711
desc='Slice-to-vol interpolation model for estimation step',
714712
requires=['mporder'],
715713
min_ver='5.0.11')
716-
slspec = traits.File(
714+
slice_order = traits.File(
717715
exists=True,
718716
argstr='--slspec=%s',
719717
desc='Name of text file completely specifying slice/group acquisition',
@@ -725,7 +723,7 @@ class EddyInputSpec(FSLCommandInputSpec):
725723
argstr='--json=%s',
726724
desc='Name of .json text file with information about slice timing',
727725
requires=['mporder'],
728-
xor=['slspec'],
726+
xor=['slice_order'],
729727
min_ver='6.0.1')
730728

731729
estimate_move_by_susceptibility = traits.Bool(
@@ -856,7 +854,7 @@ class Eddy(FSLCommand):
856854
>>> eddy.inputs.in_bval = 'bvals.scheme'
857855
>>> eddy.inputs.use_cuda = True
858856
>>> eddy.cmdline # doctest: +ELLIPSIS
859-
'eddy_cuda --flm=quadratic --ff=10.0 --fwhm=0.0 \
857+
'eddy_cuda --flm=quadratic --ff=10.0 \
860858
--acqp=epi_acqp.txt --bvals=bvals.scheme --bvecs=bvecs.scheme \
861859
--imain=epi.nii --index=epi_index.txt --mask=epi_mask.nii \
862860
--interp=spline --resamp=jac --niter=5 --nvoxhp=1000 \
@@ -865,7 +863,7 @@ class Eddy(FSLCommand):
865863
Running eddy on a CPU using OpenMP:
866864
>>> eddy.inputs.use_cuda = False
867865
>>> eddy.cmdline # doctest: +ELLIPSIS
868-
'eddy_openmp --flm=quadratic --ff=10.0 --fwhm=0.0 \
866+
'eddy_openmp --flm=quadratic --ff=10.0 \
869867
--acqp=epi_acqp.txt --bvals=bvals.scheme --bvecs=bvecs.scheme \
870868
--imain=epi.nii --index=epi_index.txt --mask=epi_mask.nii \
871869
--interp=spline --resamp=jac --niter=5 --nvoxhp=1000 \
@@ -874,17 +872,17 @@ class Eddy(FSLCommand):
874872
Running eddy with slice-to-volume motion correction:
875873
>>> eddy.inputs.use_cuda = True
876874
>>> eddy.inputs.mporder = 6
877-
>>> eddy.inputs.s2v_niter = 5
878-
>>> eddy.inputs.s2v_lambda = 1
879-
>>> eddy.inputs.s2v_interp = 'trilinear'
880-
>>> eddy.inputs.slspec = 'epi_slspec.txt'
875+
>>> eddy.inputs.slice2vol_niter = 5
876+
>>> eddy.inputs.slice2vol_lambda = 1
877+
>>> eddy.inputs.slice2vol_interp = 'trilinear'
878+
>>> eddy.inputs.slice_order = 'epi_slspec.txt'
881879
>>> eddy.cmdline # doctest: +ELLIPSIS
882-
'eddy_cuda --flm=quadratic --ff=10.0 --fwhm=0.0 \
880+
'eddy_cuda --flm=quadratic --ff=10.0 \
883881
--acqp=epi_acqp.txt --bvals=bvals.scheme --bvecs=bvecs.scheme \
884882
--imain=epi.nii --index=epi_index.txt --mask=epi_mask.nii \
885883
--interp=spline --resamp=jac --mporder=6 --niter=5 --nvoxhp=1000 \
886-
--out=.../eddy_corrected --s2v_interp=trilinear --s2v_niter=5 \
887-
--slm=none --slspec=epi_slspec.txt'
884+
--out=.../eddy_corrected --s2v_interp=trilinear --s2v_lambda=1 \
885+
--s2v_niter=5 --slspec=epi_slspec.txt --slm=none'
888886
>>> res = eddy.run() # doctest: +SKIP
889887
890888
"""

nipype/interfaces/fsl/tests/test_auto_Eddy.py

Lines changed: 25 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,7 @@ def test_Eddy_inputs():
3737
argstr='--ff=%s',
3838
usedefault=True,
3939
),
40-
fwhm=dict(
41-
argstr='--fwhm=%s',
42-
usedefault=True,
43-
),
40+
fwhm=dict(argstr='--fwhm=%s', ),
4441
in_acqp=dict(
4542
argstr='--acqp=%s',
4643
extensions=None,
@@ -93,16 +90,7 @@ def test_Eddy_inputs():
9390
argstr='--json=%s',
9491
min_ver='6.0.1',
9592
requires=['mporder'],
96-
xor=['slspec'],
97-
),
98-
mb=dict(
99-
argstr='--mb=%s',
100-
min_ver='5.0.10',
101-
),
102-
mb_offs=dict(
103-
argstr='--mb_offs=%s',
104-
min_ver='5.0.10',
105-
requires=['mb'],
93+
xor=['slice_order'],
10694
),
10795
mbs_ksp=dict(
10896
argstr='--mbs_ksp=%smm',
@@ -128,6 +116,15 @@ def test_Eddy_inputs():
128116
min_ver='5.0.11',
129117
requires=['use_cuda'],
130118
),
119+
multiband_factor=dict(
120+
argstr='--mb=%s',
121+
min_ver='5.0.10',
122+
),
123+
multiband_offset=dict(
124+
argstr='--mb_offs=%d',
125+
min_ver='5.0.10',
126+
requires=['multiband_factor'],
127+
),
131128
niter=dict(
132129
argstr='--niter=%s',
133130
usedefault=True,
@@ -175,35 +172,35 @@ def test_Eddy_inputs():
175172
argstr='--residuals',
176173
min_ver='5.0.10',
177174
),
178-
s2v_interp=dict(
175+
session=dict(
176+
argstr='--session=%s',
177+
extensions=None,
178+
),
179+
slice2vol_interp=dict(
179180
argstr='--s2v_interp=%s',
180181
min_ver='5.0.11',
181182
requires=['mporder'],
182183
),
183-
s2v_lambda=dict(
184-
agstr='--s2v_lambda',
184+
slice2vol_lambda=dict(
185+
argstr='--s2v_lambda=%d',
185186
min_ver='5.0.11',
186187
requires=['mporder'],
187188
),
188-
s2v_niter=dict(
189-
argstr='--s2v_niter=%s',
189+
slice2vol_niter=dict(
190+
argstr='--s2v_niter=%d',
190191
min_ver='5.0.11',
191192
requires=['mporder'],
192193
),
193-
session=dict(
194-
argstr='--session=%s',
195-
extensions=None,
196-
),
197-
slm=dict(
198-
argstr='--slm=%s',
199-
usedefault=True,
200-
),
201-
slspec=dict(
194+
slice_order=dict(
202195
argstr='--slspec=%s',
203196
min_ver='5.0.11',
204197
requires=['mporder'],
205198
xor=['json'],
206199
),
200+
slm=dict(
201+
argstr='--slm=%s',
202+
usedefault=True,
203+
),
207204
use_cuda=dict(),
208205
)
209206
inputs = Eddy.input_spec()

0 commit comments

Comments
 (0)