Skip to content

[DOC] update FSL preprocess #1881

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
Mar 16, 2017
Merged

[DOC] update FSL preprocess #1881

merged 4 commits into from
Mar 16, 2017

Conversation

neuroumbrage
Copy link
Contributor

@satra Does this work?

# Conflicts:
#	nipype/algorithms/tests/test_auto_NonSteadyStateDetector.py
@satra
Copy link
Member

satra commented Mar 14, 2017

yes - much cleaner. thank you.

super(ApplyXfm, self).__init__(**inputs)
warn(('This interface has been renamed since 0.12.1, please use '
'nipype.interfaces.fsl.ApplyXFM'),
UserWarning)
Copy link
Member

Choose a reason for hiding this comment

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

This was removed in #1866. Looks like an accidental re-add.

@@ -908,6 +923,9 @@ class FNIRTOutputSpec(TraitedSpec):
class FNIRT(FSLCommand):
"""Use FSL FNIRT for non-linear registration.

For complete details, see the `MCFLIRT Documentation.
Copy link
Member

Choose a reason for hiding this comment

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

MCFLIRT => FNIRT

@@ -1208,6 +1226,9 @@ class SUSANOutputSpec(TraitedSpec):
class SUSAN(FSLCommand):
""" use FSL SUSAN to perform smoothing

For complete details, see the `MCFLIRT Documentation.
Copy link
Member

Choose a reason for hiding this comment

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

Likewise, MCFLIRT => SUSAN.

@codecov-io
Copy link

codecov-io commented Mar 14, 2017

Codecov Report

Merging #1881 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #1881   +/-   ##
=======================================
  Coverage   73.11%   73.11%           
=======================================
  Files        1061     1061           
  Lines       53375    53375           
=======================================
  Hits        39025    39025           
  Misses      14350    14350
Flag Coverage Δ
#unittests 73.11% <100%> (ø)
Impacted Files Coverage Δ
nipype/interfaces/fsl/preprocess.py 77.38% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd1ed4f...f5f1b22. Read the comment docs.

@satra satra merged commit a99e02d into nipy:master Mar 16, 2017
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.

4 participants