Skip to content

Laplace-Beltrami on surfaces #342

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

Closed
wants to merge 2 commits into from
Closed

Laplace-Beltrami on surfaces #342

wants to merge 2 commits into from

Conversation

jordandekraker
Copy link
Collaborator

Using the Laplace-Beltrami operator on a surface is much faster than a finite-differences solution on a volume, especially at very high resolutions.

Eventually I would like to apply this to levelset surfaces generated directly by nnunet to avoid having to use the Laplace solver entirely. This has the secondary advantage of solving "holes" and "bridges" in the SRLM and alveus segmentations, allowing us to eliminate shape_inject.smk. Using Laplace-Beltrami on top of this would be a large speed boost, but even keeping just the IO Laplace solver is still an improvement since AP is the slowest to solve (furthest distances).

Notes:

  • native surfaces should have no holes, and must have only one connected-component
  • the current surface edge detector is sensitive to spikey edges

I hope to clean up both of these issues by playing around with levelsets

This logic is shown in one notebook (for illustration only). This should eventually replace autotop.smk (possibly keeping the IO equivolumetric solver) and warps.smk

@akhanf
Copy link
Member

akhanf commented Jan 3, 2025

Cool looks promising! I also agree that IO coords volumetrically then LB on surface could be a good way forward.

Probably lots to catch up on in Tues call - I have a PR that has improvements to native surf along with refactored unfolded registration - hipp surfaces are looking great now with no holes or adhoc fixes, and should work well with this. PR is just about ready hope to put a draft up this weekend.

@jordandekraker
Copy link
Collaborator Author

taken over by #353

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