Add moving least squares functions for pointclouds and surfaces

haesleinhuepf / napari-process-points-and-surfaces

Process and analyze surfaces using vedo in napari.

BSD 3-Clause "New" or "Revised" License

21 stars 4 forks source link

Add moving least squares functions for pointclouds and surfaces #65

Closed jo-mueller closed 1 year ago

jo-mueller commented 1 year ago

This PR partially addresses #64.

It adds the functions smooth_surface_moving_least_squares_2D, smooth_surface_moving_least_squares_2D_radius, smooth_pointcloud_moving_least_squares_2D and smooth_pointcloud_moving_least_squares_2D_radius.

Steps to verify quality:

[x] Automated tests are passing
[x] Tested all functions manually in the viewer
[x] Added functions to demo notebook

haesleinhuepf commented 1 year ago

Awesome, thanks!

haesleinhuepf commented 1 year ago

Just a minor remark. My IDE proposes this:

jo-mueller commented 1 year ago

@haesleinhuepf

you mean regarding the line "The radius is used to determine the number of points [...]"? I could change the points to vertices? That would be technically more correct.

haesleinhuepf commented 1 year ago

@jo-mueller I'm referring to the error message Function name should be lowercase.