Skip to content

fix: make dcm2nii compatible with node for efficient execution, improve docs #782

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
Feb 1, 2014

Conversation

satra
Copy link
Member

@satra satra commented Jan 31, 2014

these changes allow dcm2nii to only convert the relevant files instead of having to read everything.


>>> from nipype.interfaces.dcm2nii import Dcm2nii
>>> converter = Dcm2nii()
>>> converter.inputs.source_names = 'functional_1.dcm'
Copy link
Member

Choose a reason for hiding this comment

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

Isn't this example a bit misleading? In most cases one would input a list of dicom files (unless they want to convert on 3D DICOM file).

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling d0e0990 on satra:enh/dcm2nii into a281843 on nipy:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 12c03b8 on satra:enh/dcm2nii into a281843 on nipy:master.

* upstream/master:
  updated changelog
  0.9.2 release
  cleanup
  Added VtoMat vista interface and updated CHANGES and autogenerated tests accordingly
  added test for Vnifti2Image interface
  updated CHANGES log
  Created interface for vimage2nifti command from lipvia vista package
  fixed mcflirt mean image output
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 8aa9aa8 on satra:enh/dcm2nii into b1b7825 on nipy:master.

chrisgorgo added a commit that referenced this pull request Feb 1, 2014
fix: make dcm2nii compatible with node for efficient execution, improve docs
@chrisgorgo chrisgorgo merged commit cb0dd44 into nipy:master Feb 1, 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