Skip to content

Conversation

hjmjohnson
Copy link
Contributor

The antsRegistration program does not have a
named sampling strategy called "Dense". In
order to get the 1 sample per voxel sampling
strategy (aka dense sampling) the user must
either not supply anything, or explicitly
supply the "None" name.

Perhaps this should have been called the
"subSamplingStrategy", but that is too
invasive of a change to ants.

The antsRegistration program does not have a
named sampling strategy called "Dense".  In
order to get the 1 sample per voxel sampling
strategy (aka dense sampling) the user must
either not supply anything, or explicitly
supply the "None" name.

Perhaps this should have been called the
"subSamplingStrategy", but that is too
invasive of a change to ants.
@hjmjohnson
Copy link
Contributor Author

@satra This patch set has only the 1 change that I made, and it is not confounded with your/chris previous 5 changes.

satra added a commit that referenced this pull request Nov 19, 2013
…tegy

BUG: antsRegistration command line failure
@satra satra merged commit c9c34c1 into nipy:master Nov 19, 2013
@hjmjohnson
Copy link
Contributor Author

@satra Thanks for pulling this in. I appreciate your attention.

@hjmjohnson hjmjohnson deleted the FixantsRegistrationSamplingStrategy branch March 14, 2014 14:00
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