OpenCascade/occt
1
0
Fork 0
mirror of https://git.dev.opencascade.org/repos/occt.git synced 2025-08-04 13:13:25 +03:00
Code Activity

0029400: Fuse of two edges creates self-interfered shape

Browse Source 
Avoid creation of the edges with too small valid range (not allowing to split the edge) and interfering vertices.
Test case for the issue.
This commit is contained in:
emv
2017-12-25 11:47:53 +03:00
parent 8197951d7e
commit e25185fff0
2 changed files with 47 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
src/BOPAlgo/BOPAlgo_PaveFiller_2.cxx
View File

@@ -392,21 +392,42 @@ void BOPAlgo_PaveFiller::SplitPaveBlocks(const TColStd_MapOfInteger& theMEdges,
myDS->UpdatePaveBlockWithSDVertices(aPBN);
FillShrunkData(aPBN);
//
if (!aPBN->HasShrunkData()) {
// No valid range, unify vertices
Standard_Boolean bHasValidRange = aPBN->HasShrunkData();
// Take into account that the edge could have really small valid range,
// so that the Pave Block cannot be further split. In this case, check if
// the vertices of the Pave Block do not interfere. And if they are, unify them.
Standard_Boolean bCheckDist = (bHasValidRange && !aPBN->IsSplittable());
if (!bHasValidRange || bCheckDist)
{
Standard_Integer nV1, nV2;
aPBN->Indices(nV1, nV2);
if (nV1 != nV2) {
if (nV1 == nV2)
// Same vertices -> no valid range, no need to unify vertices
continue;
// Decide whether to unify vertices or not
if (bCheckDist)
{
const TopoDS_Vertex& aV1 = TopoDS::Vertex(myDS->Shape(nV1));
const TopoDS_Vertex& aV2 = TopoDS::Vertex(myDS->Shape(nV2));
if (BOPTools_AlgoTools::ComputeVV(aV1, aV2, myFuzzyValue) == 0)
// vertices are interfering -> no valid range, unify vertices
bHasValidRange = Standard_False;
}
if (!bHasValidRange)
{
BOPDS_Pair aPair;
aPair.SetIndices(nV1, nV2);
if (aMPairs.Add(aPair)) {
if (aMPairs.Add(aPair))
{
TColStd_ListOfInteger aLV;
aLV.Append(nV1);
aLV.Append(nV2);
MakeSDVertices(aLV, theAddInterfs);
}
continue;
}
continue;
}
//
// Update the list with new pave block