Skip to content

Additional dcm2nii inputs #931

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 4 commits into from
Sep 30, 2014
Merged

Additional dcm2nii inputs #931

merged 4 commits into from
Sep 30, 2014

Conversation

mick-d
Copy link
Contributor

@mick-d mick-d commented Sep 25, 2014

No description provided.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) when pulling e639b87 on mick-d:master into f065d9c on nipy:master.

reorient = traits.Bool(argstr='-r', position=11)
spm_analyze = traits.Bool(argstr='-s', xor=['nii_output'], position=12)
convert_all_pars = traits.Bool(True, argstr='-v', usedefault=True, position=13)
reorient_and_crop = traits.Bool(False, argstr='-x', usedefault=True, position=14)
args = traits.Str(argstr='%s', desc='Additional parameters to the command',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is defined in CommandLineInputSpec it could be removed from here

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove args line? Ok

@mick-d
Copy link
Contributor Author

mick-d commented Sep 26, 2014

Hi Chris (not Greg 😃 ) Does it seem fine now?

@coveralls
Copy link

Coverage Status

Coverage increased (+0.0%) when pulling a686236 on mick-d:master into f065d9c on nipy:master.

@mick-d
Copy link
Contributor Author

mick-d commented Sep 26, 2014

Actually i'll add some documentation so i'll submit an update in a bit

@mick-d
Copy link
Contributor Author

mick-d commented Sep 26, 2014

Help() function gives good enough doc so i believe this is ready for merging

chrisgorgo added a commit that referenced this pull request Sep 30, 2014
Additional dcm2nii inputs
@chrisgorgo chrisgorgo merged commit 25cb849 into nipy:master Sep 30, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants