CadQuery / cadquery

A python parametric CAD scripting framework based on OCCT
https://cadquery.readthedocs.io
Other
2.93k stars 276 forks source link

Fix crash on invalid faces #1541

Closed openvmp closed 1 month ago

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.86%. Comparing base (153ed3f) to head (596087d). Report is 11 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1541 +/- ## ========================================== + Coverage 94.48% 94.86% +0.38% ========================================== Files 28 28 Lines 5780 6228 +448 Branches 1071 1262 +191 ========================================== + Hits 5461 5908 +447 Misses 193 193 - Partials 126 127 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

adam-urbanczyk commented 1 month ago

@openvmp thanks, what is the background of this fix? How can I reproduce the issue and can a test be added?

openvmp commented 1 month ago

I was trying to use CadQuery to render models for PartCAD.org. Specifically this model: https://partcad.org/repository/assembly/robotics/multimodal/openvmp/robots/don1:robot

It was larger than 1GB back then. And CadQuery was crashing on some triangle. It was too much for me to find which one and why. I stopped using CadQuery for that purpose, so it's impossible to investigate any further now.

adam-urbanczyk commented 1 month ago

Could you share this model as a .brep file?

openvmp commented 1 month ago

Give me some time to add this export feature and to try reproducing the crash (I operate on much smaller models now).

adam-urbanczyk commented 1 month ago

@openvmp I think I found a MRE:

cq.Face.makePlane(1e-9,1e-9).tessellate(1e-3)
adam-urbanczyk commented 1 month ago

Yes, me too.

adam-urbanczyk commented 1 month ago

@jmwright are you OK with merging this PR?

jmwright commented 1 month ago

@adam-urbanczyk +1 to merge

adam-urbanczyk commented 1 month ago

Thanks @openvmp !