-
Notifications
You must be signed in to change notification settings - Fork 315
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
Verification on other Data Sets #3
Comments
Hi, great question - thanks :) So ModelNet40 / ShapeNet is not manifold. Meshes that are not manifold do not have at most 2 incident faces - which breaks our fixed-size convolution neighborhood assumption (illustration) I did start playing with this code which makes meshes manifold, and noticed it works pretty well. I will try to find some time soon to write some scripts to clean it up and add it to the repo. Will update this issue accordingly |
It might be worth using something like This would also make importing datasets of different formats easier as well, as loading a mesh would be something like The function
The above code is completely untested... |
@mrmoss I think the neural network would need a mesh with a single (or only a few) components. What pymesh and trimesh do for making the mesh manifold is breaking it into multiple components. So, they might not work that easily (I am not sure). |
I was thinking more detection and warning/erroring rather than repairing. As far as I can gather from both mentioned modules, other than face normals, files are parsed without any attempt to repair or fix them. For repairing, trimesh's fill_holes() might be worth a try? Edit: Edited for overall clarity and length. |
I tried these non-manifold fixes, and they do not work for shapenet.
I do hear what you are saying, but I prefer to keep the code independent of external packages, and just have people convert to .obj format. This can be done pretty easily with Meshlab scripts. And maybe it's just me, but I think when things are written explicitly, it helps make it really clear how simple meshes really are :) |
These guys seem to have processed all the ShapeNet. |
Has anyone by now tried the network on the ModelNet50/ModelNet10 (classification) dataset? |
There is a dataset Manifold40, all shapes are manifold. |
This is more a question rather than an issue, so apologies if it is not the right place to ask it.
Have you also tried the network on the de facto ModelNet50 (classification) and ShapeNet Core (segmentation) datasets?
The text was updated successfully, but these errors were encountered: