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

Mesh repair #9

Closed
cdcseacave opened this issue May 8, 2019 · 5 comments
Closed

Mesh repair #9

cdcseacave opened this issue May 8, 2019 · 5 comments

Comments

@cdcseacave
Copy link

First of all thank you for sharing this great library.

I am developing an photogrammetry module (OpenMVS) and so I am using meshes at very low level. While this library would be enough for 99% of my needs, it has one feature missing, like most of the mesh libraries out there: repair a non-manifold mesh. During MVS reconstruction it is possible to get meshes with few non-manifold vertices/edges, so before doing any further processing I need to fix them (preferably not by simply deleting them, but by repairing: add a new vertex for non-manifold vertices, etc).

Is this library supporting this? I'm not able to find it. If I'd had that feature I will immediately switch OpenMVS to use it instead of VCG, which offers a lot of functionality, but has a very strange structure and many bugs.

@dsieger
Copy link
Member

dsieger commented May 26, 2019

We actually thought about adding a feature like this. However, this is unlikely to arrive soon.

@dsieger dsieger closed this as completed May 26, 2019
@mbotsch mbotsch reopened this May 26, 2019
@mbotsch
Copy link
Member

mbotsch commented May 26, 2019

Let me add one question: Would it be sufficient for you to simply duplicate the non-manifold vertices? This would let to a manifold mesh, but with possibly quite some isolated faces. Hence further processing (e.g. remeshing) might not lead to the extected results. This duplication could be done quite easily.

What Daniel was mentioning is a proper representation of non-manifold situation that also allows for processing and optimization. This might take us quite some time...

@cdcseacave
Copy link
Author

That helps, of course, but it is not ideal and also will not motivate me to invest time in replacing an imperfect mesh processing library with an other imperfect one.

@mbotsch
Copy link
Member

mbotsch commented May 26, 2019

Ok. Good luck finding the perfect one ;-)

@mbotsch mbotsch closed this as completed May 26, 2019
@cdcseacave
Copy link
Author

thx, will do!

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