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

Added installation hints and removed duplicated vertices in notebook #50

Closed jo-mueller closed 1 year ago

jo-mueller commented 1 year ago

Hi @haesleinhuepf ,

I just tested all the notebooks and they ran fine. One exception: creating_surfaces.ipynb I cannot run because it requires some intensity image data on a local drive.

Just two minor things I added:

haesleinhuepf commented 1 year ago

Awesome, thanks Johannes @jo-mueller !

I added surface = remove_duplicate_vertices(surface) in the quality_measurements.ipynb because some weird artefacts appeared in the smoothed surface data.

Might this go away after fixing #48 ?

jo-mueller commented 1 year ago

Might this go away after fixing https://github.com/haesleinhuepf/napari-process-points-and-surfaces/issues/48 ?

It should, yes. It's more in the interest of getting the vedo/open3d conversion out that I'd prefer a new release sometimes soon and fixing other issues later-on.