Skip to content

Update and Clean ants.segmentation #990

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 9 commits into from
Nov 8, 2014
Merged

Update and Clean ants.segmentation #990

merged 9 commits into from
Nov 8, 2014

Conversation

blakedewey
Copy link
Contributor

Updated input spec and arg formatting for N4BiasFieldCorrection and cleaned up ants.segmentation for PEP8

blakedewey added 4 commits November 6, 2014 10:26
Added weight_image to input spec. Added bspline_order to input spec and
updated arg formatting for bspline_fitting_distance to allow for
bspline_order to be set. Updated n_iterations and convergence_threshold
arg formatting to conform with other arg formatting for this class.
Gave ants.segmentation some PEP8 love. Also updated _run_interface call
to include correct_return_codes which is declared in the overridden
function
@satra
Copy link
Member

satra commented Nov 6, 2014

could you please also update the CHANGES file as well?

@blakedewey
Copy link
Contributor Author

Updated as an ENH

@@ -1,6 +1,11 @@
Next release
============

* ENH: Updated N4BiasCorrection input spec to include weight image and spline order. Made
argument formatting consistent. Cleaned ants.segmentation according to PEP8.
Copy link
Member

Choose a reason for hiding this comment

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

could you please add the link to the PR here as well?

https://github.com/nipy/nipype/pull/990/files

@blakedewey
Copy link
Contributor Author

Done!

satra added a commit that referenced this pull request Nov 8, 2014
Update and Clean ants.segmentation
@satra satra merged commit 762c08c into nipy:master Nov 8, 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.

2 participants