-
Notifications
You must be signed in to change notification settings - Fork 95
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
Bug in minConvexHull ? #27
Comments
hi, not really a bug, but the output of minConvexHull does not impose faces to have consistent orientation. This would be interesting, however... |
Hi, I have just made a commit that should solve the problem. You can check result of face orientation with the following: Hope this solves the problem? |
Thank you, I will evalute it and give feedback. |
There is still an error, if you remove the last vertex of the testVertices.mat from above :
Thanks in advance |
Hi, |
Hi, |
I'll check it |
Seems to work. However, one should be aware that unreferenced vertices can remain:
|
Hi, |
Hi
I've tried minConvexHull, but I get a mesh with edges with two faces on one side.
Here is the test case:
test_minConvexHull.zip
One of the bad edges is marked in red here:
Thanks in advance
The text was updated successfully, but these errors were encountered: