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

Taubin's mesh smoothing algorithm. #26

Open
wants to merge 2 commits into
base: connected-components
Choose a base branch
from

Conversation

tinevez
Copy link
Member

@tinevez tinevez commented May 26, 2023

Adapted by the Javascript code of mykolalysenko, MIT license. https://github.com/mikolalysenko/taubin-smooth/blob/master/smooth.js

Taubin, G. “Curve and Surface Smoothing without Shrinkage.” In Proceedings of IEEE International Conference on Computer Vision, 852–57, 1995.

Screenshot 2023-05-26 at 11 04 47

From left to right:

  • smoothing with naive weights, 10 iterations, lambda = 0.5, mu = -0.53 (like MeshLab)
  • original noisy cube
  • smoothing with cotangent weights, 10 iterations, passband = 0.1

tinevez and others added 2 commits May 25, 2023 22:23
Adapted by the Javascript code of mykolalysenko, MIT license.
https://github.com/mikolalysenko/taubin-smooth/blob/master/smooth.js
This implementation is not satisfactory. I could not reproduce the
results of MeshLab, and it has some small instabilities for iter = 50.
@ctrueden
Copy link
Member

@tinevez This is great! 😄

But I'm confused: is this PR ready for review+merge, or not? If it's not ready, it should be marked draft. If it is ready, there should not be any WIP commits...

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