OpenCascade/occt
1
0
Fork 0
mirror of https://git.dev.opencascade.org/repos/occt.git synced 2025-04-03 17:56:21 +03:00
Code Packages Releases Activity

0032692: Mesh - In BRepMesh_ModelPreProcessor avoid crashes with problematic topology

Browse Source 
Improved the processing for the case where Failure flag for the meshing face is set. Specific tests added.
Check triangulation consistency even for shapes with problematic topology given that they are suitable for keeping it.
This commit is contained in:
oan 2022-06-28 11:43:49 +03:00 committed by smoskvin
parent aaa03c1efb
commit 1593d38b13
4 changed files with 41 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/BRepMesh/BRepMesh_ModelPreProcessor.cxx
View File

@ -46,7 +46,8 @@ namespace
void operator()(const Standard_Integer theFaceIndex) const
{
const IMeshData::IFaceHandle& aDFace = myModel->GetFace(theFaceIndex);
if (aDFace->IsSet(IMeshData_Outdated))
if (aDFace->IsSet(IMeshData_Outdated) ||
aDFace->GetFace().IsNull())
{
return;
}
@ -119,7 +120,7 @@ namespace
void operator()(const Standard_Integer theFaceIndex) const
{
const IMeshData::IFaceHandle& aDFace = myModel->GetFace(theFaceIndex);
if (aDFace->GetSurface()->GetType() != GeomAbs_Cone)
if (aDFace->GetSurface()->GetType() != GeomAbs_Cone || aDFace->IsSet(IMeshData_Failure))
{
return;
}
@ -128,7 +129,7 @@ namespace
for (Standard_Integer aEdgeIdx = 0; aEdgeIdx < aDWire->EdgesNb() - 1; ++aEdgeIdx)
{
const IMeshData::IEdgePtr& aDEdge = aDWire->GetEdge (aEdgeIdx);
if (aDEdge->GetPCurve(aDFace.get(), TopAbs_FORWARD) != aDEdge->GetPCurve(aDFace.get(), TopAbs_REVERSED))
{
if (aDEdge->GetCurve()->ParametersNb() == 2)
@ -146,7 +147,7 @@ namespace
}
}
return;
}
}
}
}

11
tests/bugs/mesh/bug32692_1 Normal file
View File

@ -0,0 +1,11 @@
puts "========="
puts "OCC32692: Crash when the input shape has some invalid topology"
puts "========="
puts ""
puts "REQUIRED ALL: Meshing statuses: SelfIntersectingWire Failure"
restore [locate_data_file bug32692.brep] s
incmesh s 0.01 -parallel
checktrinfo s -nod 7511 -tri 7625 -empty 9 -face 309

11
tests/bugs/mesh/bug32692_2 Normal file
View File

@ -0,0 +1,11 @@
puts "========="
puts "OCC32692: Crash when the input shape has some invalid topology"
puts "========="
puts ""
puts "REQUIRED ALL: Meshing statuses: Reused"
restore [locate_data_file bug32692_broken_mesh.brep] s
incmesh s 0.01 -parallel
checktrinfo s -nod 3 -tri 1 -face 1

14
tests/bugs/mesh/bug32692_3 Normal file
View File

@ -0,0 +1,14 @@
puts "========="
puts "OCC32692: Crash when the input shape has some invalid topology"
puts "========="
puts ""
puts "REQUIRED ALL: Meshing statuses: OpenWire Failure Outdated"
pload MODELING
cylinder c 0 0 0 10
mkface f c
incmesh f 0.01 -parallel
checktrinfo f -nod 0 -tri 0 -face 1