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

unreferenced vertex of point cloud #1

Open
feiran-l opened this issue Aug 28, 2020 · 2 comments
Open

unreferenced vertex of point cloud #1

feiran-l opened this issue Aug 28, 2020 · 2 comments

Comments

@feiran-l
Copy link

Hi, thank you for sharing the nice work!

I was trying to construct the laplacian from the point clouds. However, it some times raise RuntimeError that GC_SAFETY_ASSERT FAILURE from /tmp/pip-req-build-hgag1o29/deps/geometry-central/src/surface/surface_mesh.cpp:139 - unreferenced vertex 377. For some point clouds if I re-run the code it will then process smoothly, but for others it will always stuck at the error. Is there anything I can do the avoid this problem?

Here is a sample for test: ply.zip.

@nmwsharp
Copy link
Owner

nmwsharp commented Sep 3, 2020

Thanks for reaching out!

I think here you're running in to the issues mentioned in the "known limitations" in the README. This is totally just a software problem because I used simple libraries and data handling for this code release :) I'm going to try to release a v2 soon which addresses this, though it might be a few days.

One possible hacky immediate fix is to add a small random jitter to your points before computing the Laplacian; this should remove the degenerate cases which are breaking the 2D Delaunay implementation. But obviously that's not super satisfying. I'll comment when I've had a chance to write some more code!

@brucedjones
Copy link
Contributor

I'm seeing this issue in the MeshHeatMethodDistance function, heres a snippet from the stack trace:

self.bound_solver = pp3db.MeshHeatMethodDistance(V, F, t_coef, use_robust)\
RuntimeError: GC_SAFETY_ASSERT FAILURE from /tmp/pip-req-build-biwr300e/deps/geometry-central/src/surface/surface_mesh.cpp:139 - unreferenced vertex 21879\

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

No branches or pull requests

3 participants