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

WIP: Add xfrm option for BRATs pipeline. #407

Merged
merged 3 commits into from
Nov 17, 2022
Merged

WIP: Add xfrm option for BRATs pipeline. #407

merged 3 commits into from
Nov 17, 2022

Conversation

ntustison
Copy link
Member

** Do not merge **

Using for evaluation purposes.

@ntustison
Copy link
Member Author

Hey @stnava , what do you think about the additional flexibility allowed by this commit? Given that I finished the BRATs evaluation, should we keep it? As a reminder, it would allow you to change the deformable metric parameter for the antsRegistrationSyN* options, e.g., radius of 2 for CC or number of bins for MI metric. I'd get rid of the "brats" transform before merging, though.

@stnava
Copy link
Member

stnava commented Nov 16, 2022 via email

@ntustison
Copy link
Member Author

Okay, will do. And I'll also add the same thing to ANTsR.

@ntustison ntustison merged commit 5aeb47b into master Nov 17, 2022
@ncullen93 ncullen93 deleted the WipBratsDi branch February 15, 2024 23:46
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