carve icon indicating copy to clipboard operation
carve copied to clipboard

Fix infinite loop bug in mergeFacesAndHoles

Open birgerbr opened this issue 6 years ago • 2 comments

When an element in containing_faces has a size greater than 1 the while (unassigned) loop will never stop.

I don't know this algorithm and I'm not sure if this is the correct way of handling this case, but it solves the infinite loop case.

birgerbr avatar Aug 29 '19 11:08 birgerbr

The case was when using ifcplusplus to extract the geometry from an IFC file from a customer. Unfortunately, I could not find any notes saying which IFC file it was.

birgerbr avatar Jun 22 '20 06:06 birgerbr

I am interested in this issue. I have observed infinite loops in carve sometimes, not sure if it is related to this. I tried the proposed fix and tested it with https://github.com/arnholm/angelcad-samples, but this proposed carve fix caused 2 samples to fail, so I think it is not 100% correct. PS. thank you for the carve library.

arnholm avatar Jun 29 '20 12:06 arnholm