Skip to content
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

Dlee/apply transform #19

Draft
wants to merge 105 commits into
base: master
Choose a base branch
from
Draft

Conversation

mdhe1248
Copy link
Collaborator

@mdhe1248 mdhe1248 commented Oct 7, 2024

Hi Tim,

There are quite some changes.
When you have some time, would you check this out? First of all, I did not have time to work on binary part, so this code is not ready to merge. It would not work without a separate installation of ANTs. Nonetheless, it works as I intended, and I thought it is time to work a little bit on the binary part.

If I understand correctly, the old ANTs seems to have only a single registration step (I am not so sure if it was true. May be I misunderstood). The newer ANTs is split into two steps: registration and transformation. I applied this to my Julia code. This is convenient when one needs to apply the same transformations multiple times.

As I see in the previous pull request( #13 ), it would be great if the output transformation from ANTs registration matches with that of CoordinateTransformation. I did not work on that thought. I may at some point, but not right now.

mdhe1248 and others added 30 commits September 26, 2024 23:49
add initial moving transform, initial fixed transform, and restrict deformation options
type error abstractTransformation -> AbstractTransformation
Syntax error: dimension and eltype
…ed values because it is too long to display all values.
…he case of SyN, it returns warp and inversewarp.
@timholy timholy self-assigned this Oct 23, 2024
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