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.

Sign up for GitHub

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

Jump to bottom

Improve fieldmap-less including laplacians for the cost-function #186

Closed
oesteban opened this issue Mar 5, 2021 · 0 comments
Closed

Improve fieldmap-less including laplacians for the cost-function #186

oesteban opened this issue Mar 5, 2021 · 0 comments
Labels
effort: medium Estimated medium effort task enhancement New feature or request Gathering feedback We would like to hear from the community about this particular issue - What are your thoughts? impact: medium Estimated medium impact task T1w-SyN
Milestone
2.1.0

Comments

@oesteban
Copy link
Member

oesteban commented Mar 5, 2021

No description provided.

@oesteban oesteban added enhancement New feature or request T1w-SyN impact: medium Estimated medium impact task effort: medium Estimated medium effort task Gathering feedback We would like to hear from the community about this particular issue - What are your thoughts? labels Mar 5, 2021
@oesteban oesteban added this to the 2.1.0 milestone Oct 1, 2021
oesteban added a commit that referenced this issue Oct 4, 2021
@oesteban
enh: continuing deep overhaul of fieldmapless implementation
ec1ee59 
Salient points on this commit:

- There's no need for intensity inversion if MI is used as cost function
- Use of the T1w as reference (fixed) image increases reliability
- Giving more flexibility to ``restrict_deformation`` with 0.1 instead
  of 1 makes optimization faster leveraging a minimal increase of dofs
- To double-check things look good, use our internal transform object
  to generate the unwarped reference, and then generate the field, which
  should be fairly close to the one generated by ANTs.
- Addressed LPS orientation bug in the function node that extracts the
  nonzero component of distortion.
- Better registration parameters, with thanks to @yasseraleman for
  suggestions (in particular, the one about flexibilizing the gradient
  restriction above came from him).
- Use of laplacians in a multi-channel registration framework

Resolves: #186.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
effort: medium Estimated medium effort task enhancement New feature or request Gathering feedback We would like to hear from the community about this particular issue - What are your thoughts? impact: medium Estimated medium impact task T1w-SyN
Projects
None yet
Milestone
2.1.0
Development

No branches or pull requests
1 participant
@oesteban