Skip to content

Conversation

@hjmjohnson
Copy link
Contributor

This patch supersedes: #1024

In this patch, improved testing and backwards compatibility are preserved for the command line.

A newer version of ANTs has a command line feature
added that allows for more optimized processing in
multi-stage registrations.

ENH: Added new state saving interface to ANTs

In order to improve registration efficiency, it is important that
we can save and restore the state of a registration so taht
continuation may occur from a previous state.

ENH: Make transform writing mutually exclusive

Writing composite or non-composite transforms needs
to be mutually exclusive in the new version of
ants.
@hjmjohnson
Copy link
Contributor Author

@satra @chrisfilo

This patch incorporates ideas and discussion from : #1024

More complete testing of command line was also performed. Backwards compatibility is maintained such that if no "float" is specified, it does not show up on the command line. If it is specified, then True maps to 1 and False maps to 0 as required by ANTs.

It would be great to finally get this commit incorporated. Now that ITK bugs related to floating point transforms are fixed, using float = True is rapidly becoming the preferred method (not default) for running deformation based registrations.

Copy link
Member

Choose a reason for hiding this comment

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

i would omit usedefault

Expose --float option in ants registration.  This allows
for computations to be done in single precision rather
than double precision.
@hjmjohnson hjmjohnson self-assigned this Aug 4, 2015
hjmjohnson added a commit that referenced this pull request Aug 5, 2015
@hjmjohnson hjmjohnson merged commit 814997a into nipy:master Aug 5, 2015
@hjmjohnson
Copy link
Contributor Author

@satra All requested changes made and tests pass, so merging.

@hjmjohnson hjmjohnson deleted the 20150804_ANTsFloatTest branch January 19, 2016 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants