Skip to content

FIX: DTITK Interface #2514

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

Merged
merged 58 commits into from
Apr 15, 2018
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
58 commits
Select commit Hold shift + click to select a range
094c475
changes made by make, I think
Sep 11, 2017
f818961
Merge remote-tracking branch 'upstream/master'
Sep 11, 2017
0c28b39
fix: dtitk interface specs
satra Feb 23, 2018
b412659
Merge branch 'master' of https://github.com/nipy/nipype
kesshijordan Feb 27, 2018
6673083
Merge branch 'fix/specs' of https://github.com/satra/nipype into satr…
kesshijordan Feb 27, 2018
70d8d63
redo input for rigid registration such that default values are passed…
kesshijordan Feb 27, 2018
19f4904
fixed interface specs for Rigid and Affine
kesshijordan Feb 27, 2018
01d9666
incorporating changes to mandatory flag and specifying type of inputs
kesshijordan Mar 22, 2018
a913e6f
fixed TVAdjustVoxelspace; removed AdjustOrigin because it's redundant
kesshijordan Mar 22, 2018
1ef2ca6
TVResample fixed
kesshijordan Mar 22, 2018
50a9f1c
more modifications to get all interfaces working/standardized (not do…
kesshijordan Mar 23, 2018
6bb86e4
added cmdline to doctest, fixed ComposeXfm interface
kesshijordan Mar 25, 2018
197b793
Fixed TVtool
kesshijordan Mar 25, 2018
e670fe1
Incorporated recommendations from @effigies. Fixed TVResample interface
kesshijordan Mar 25, 2018
b1581ab
BinThresh fixed; xor added to TVAdjustVoxelSpace
kesshijordan Mar 25, 2018
db590d5
compressed sampling to tuple input, switched use of previously calcul…
kesshijordan Mar 25, 2018
f474554
affSymTensor3DVol interface fixed
kesshijordan Mar 26, 2018
32e4f02
Made use of affine not default per @effigies suggestion.
kesshijordan Mar 26, 2018
de50d4e
SVAdjustVoxelspace interface fixed
kesshijordan Mar 26, 2018
cb2b537
SVResample interface fixed
kesshijordan Mar 26, 2018
aeb2c55
removed old SVResample
kesshijordan Mar 26, 2018
5049458
minor wording
kesshijordan Mar 26, 2018
98e181a
affSymTensor3DVol interface fixed
kesshijordan Mar 26, 2018
f601c68
diffeoSymTensor3DVol fixed
kesshijordan Mar 26, 2018
49d0138
diffeoScalarVol interface fixed
kesshijordan Mar 26, 2018
78e155d
Merge branch 'master' of https://github.com/nipy/nipype into satra_fi…
kesshijordan Mar 26, 2018
605f3b5
From @effigies: prefatory lines no longer necessary. Tests are now ru…
kesshijordan Mar 26, 2018
ea6afb8
fix test bugs
kesshijordan Mar 27, 2018
2acb241
add tests
kesshijordan Mar 27, 2018
ddfd19d
added extra tests from other interfaces
kesshijordan Mar 27, 2018
37fb6f9
renamed interfaces; regenerated auto tests
kesshijordan Mar 28, 2018
1bc8a1c
renamed interfaces
kesshijordan Mar 28, 2018
2fea4fa
added new fake files for testing
kesshijordan Mar 28, 2018
1deaad8
Various modifications based on code review by @effigies
kesshijordan Mar 28, 2018
def5d6b
made interfaces more user friendly
kesshijordan Mar 28, 2018
5499794
fixed bug
kesshijordan Mar 29, 2018
0244983
this works
kesshijordan Mar 29, 2018
cfcb91d
fixed conversion int label to string
kesshijordan Mar 29, 2018
a10da5d
temp fix composexfm naming (still a problem)
kesshijordan Mar 29, 2018
0839823
fixed syntax, line continuation
kesshijordan Mar 30, 2018
1379dc6
fixed strings for tests
kesshijordan Mar 30, 2018
d40ba45
Merge branch 'master' of https://github.com/nipy/nipype into satra_fi…
kesshijordan Apr 3, 2018
4cedad9
merged master, ran make specs
kesshijordan Apr 3, 2018
155ae52
miscellaneous fixes from code review
kesshijordan Apr 3, 2018
e921ddf
added descriptions, few minor fixes
kesshijordan Apr 9, 2018
08babbc
removed automatic author label
kesshijordan Apr 9, 2018
f1fac40
temporary fix for composeXfm naming
kesshijordan Apr 10, 2018
cae974e
added mixin for backward compatability since interfaces were renamed
kesshijordan Apr 10, 2018
7c41d5d
fixed composeXfm naming; added tests for old interface names
kesshijordan Apr 10, 2018
5b2462c
minor fixes
kesshijordan Apr 10, 2018
a3c391c
task vs TASK
kesshijordan Apr 10, 2018
c644343
bug: copyfile=False caused execution input to be changed from the cor…
kesshijordan Apr 11, 2018
5df307c
tests for composexfm change
kesshijordan Apr 11, 2018
898db0c
made files save to node working directories
kesshijordan Apr 12, 2018
ca52d94
made check for ".aff not found" to raise error so that it doesn't fai…
kesshijordan Apr 12, 2018
6224080
changed description on initialize_xfm
kesshijordan Apr 12, 2018
769a0fc
added workflows for simple linear and nonlinear tensor registration
kesshijordan Apr 13, 2018
62e3456
added resliced ims to wf output, tweaked registration interface params
kesshijordan Apr 13, 2018
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion nipype/interfaces/afni/preprocess.py
Original file line number Diff line number Diff line change
Expand Up @@ -2510,7 +2510,7 @@ class TProjectInputSpec(AFNICommandInputSpec):
rather than the value stored in the dataset header.""",
argstr='-TR %g')
mask = File(
exist=True,
exists=True,
desc="""Only operate on voxels nonzero in the mset dataset.
++ Voxels outside the mask will be filled with zeros.
++ If no masking option is given, then all voxels
Expand Down
5 changes: 1 addition & 4 deletions nipype/interfaces/afni/tests/test_auto_TProject.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,10 +32,7 @@ def test_TProject_inputs():
mandatory=True,
position=1,
),
mask=dict(
argstr='-mask %s',
exist=True,
),
mask=dict(argstr='-mask %s', ),
noblock=dict(argstr='-noblock', ),
norm=dict(argstr='-norm', ),
num_threads=dict(
Expand Down
12 changes: 5 additions & 7 deletions nipype/interfaces/dtitk/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,8 @@
"""

# from .base import ()
from .registration import (RigidTask, AffineTask, DiffeoTask,
ComposeXfmTask, diffeoSymTensor3DVolTask,
affSymTensor3DVolTask, affScalarVolTask,
diffeoScalarVolTask)
from .utils import (TVAdjustOriginTask, TVAdjustVoxSpTask,
SVAdjustVoxSpTask, TVResampleTask, SVResampleTask,
TVtoolTask, BinThreshTask)
from .registration import (Rigid, Affine, Diffeo,
ComposeXfm, DiffeoSymTensor3DVol, AffSymTensor3DVol,
AffScalarVol, DiffeoScalarVol)
from .utils import (TVAdjustVoxSp, SVAdjustVoxSp, TVResample, SVResample,
TVtool, BinThresh)
22 changes: 22 additions & 0 deletions nipype/interfaces/dtitk/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,12 @@
* Rigid Tensor Registration
* Affine Tensor Registration
* Diffeomorphic Tensor Registration
* Combine affiine and diffeomorphic transforms
* Application of transform to tensor and scalar volumes
* Threshold and Binarize
* Adjusting the voxel space of tensor and scalar volumes
* Resampling tensor and scalar volumes
* Calculation of tensor metrics from tensor volume

Examples
--------
Expand All @@ -28,10 +34,26 @@
from ...utils.filemanip import fname_presuffix
from ..base import CommandLine
from nipype.interfaces.fsl.base import Info
import warnings

LOGGER = logging.getLogger('interface')


class DTITKRenameMixin(object):
def __init__(self, *args, **kwargs):
classes = [cls.__name__ for cls in self.__class__.mro()]
dep_name = classes[0]
rename_idx = classes.index('DTITKRenameMixin')
new_name = classes[rename_idx + 1]
warnings.warn('The {} interface has been renamed to {}\n'
'Please see the documentation for DTI-TK '
'interfaces, as some inputs have been '
'added or renamed for clarity.'
''.format(dep_name, new_name),
DeprecationWarning)
super(DTITKRenameMixin, self).__init__(*args, **kwargs)


class CommandLineDtitk(CommandLine):

def _gen_fname(self, basename, cwd=None, suffix=None, change_ext=True,
Expand Down
Loading