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 subdivision methods #25

Closed jo-mueller closed 1 year ago

jo-mueller commented 1 year ago

Hi @haesleinhuepf ,

When I ran the demo notebook with the new vedo methods, I somehow always got an empty surface layer as a result when using the vedo_subdivide_loop() function. I added the available methods to the arguments so that they show as a dropdown in napari and can still be used as the basic vedo function (int argument indicating the used method).

I thus changed the name of the function to vedo_subdivide.

Let me know what you think!

jo-mueller commented 1 year ago

Hi @haesleinhuepf ,

thanks for taking a look at it. I removed the remainders of the previous implementation in 6b1c56abd5eb50d36cbf8cd2b261a9de0042638c. The other commit (178ba56abf48c3544d63d83ea6d1965d53edcf2d) just features some auto-format that's done automatically by VSCode for Python. Hope you don't mind 😬