ForkMaster/geom
1
0
Fork 0
mirror of https://git.salome-platform.org/gitpub/modules/geom.git synced 2024-11-11 16:19:17 +05:00
Code Issues Packages Projects Releases Wiki Activity

Usage of ShapeUpgrade_UnifySameDomain instead of BlockFix_UnionFaces and BlockFix_UnionEdges.

Browse Source
This commit is contained in:
jfa 2021-09-06 15:03:21 +03:00
parent d6f16273ad
commit 8db1e932e3
2 changed files with 52 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

43
src/BlockFix/BlockFix_BlockFixAPI.cxx
View File

@ -31,6 +31,8 @@
#include <BlockFix_UnionEdges.hxx>
#include <ShapeUpgrade_RemoveLocations.hxx>
#include <TopoDS_Edge.hxx>
#include <ShapeUpgrade_UnifySameDomain.hxx>
#include <Precision.hxx>
@ -67,10 +69,31 @@ void BlockFix_BlockFixAPI::Perform()
myShape = BlockFix::RefillProblemFaces(myShape);
// faces unification
BlockFix_UnionFaces aFaceUnifier;
aFaceUnifier.GetTolerance() = myTolerance;
aFaceUnifier.GetOptimumNbFaces() = myOptimumNbFaces;
TopoDS_Shape aResult = aFaceUnifier.Perform(myShape);
TopoDS_Shape aResult = myShape;
if (myOptimumNbFaces > 1) {
// use old algo BlockFix_UnionFaces for exactly given resulting number of faces
BlockFix_UnionFaces aFaceUnifier;
aFaceUnifier.GetTolerance() = myTolerance;
aFaceUnifier.GetOptimumNbFaces() = myOptimumNbFaces;
aResult = aFaceUnifier.Perform(aResult);
}
else if (myOptimumNbFaces != -1) {
// use OCCT algo ShapeUpgrade_UnifySameDomain
ShapeUpgrade_UnifySameDomain Unifier;
//only faces
Standard_Boolean isUnifyEdges = Standard_False;
Standard_Boolean isUnifyFaces = Standard_True;
Standard_Boolean isConcatBSplines = Standard_True;
Unifier.Initialize(myShape, isUnifyEdges, isUnifyFaces, isConcatBSplines);
//Unifier.SetLinearTolerance(myTolerance);
Unifier.SetLinearTolerance(Precision::Confusion());
Unifier.SetAngularTolerance(Precision::Confusion());
Unifier.Build();
aResult = Unifier.Shape();
}
else {
// myOptimumNbFaces == -1 means do not union faces
}
// avoid problem with degenerated edges appearance
// due to shape quality regress
@ -79,8 +102,16 @@ void BlockFix_BlockFixAPI::Perform()
aResult = RemLoc.GetResult();
// edges unification
BlockFix_UnionEdges anEdgeUnifier;
myShape = anEdgeUnifier.Perform(aResult,myTolerance);
//BlockFix_UnionEdges anEdgeUnifier;
//myShape = anEdgeUnifier.Perform(aResult,myTolerance);
ShapeUpgrade_UnifySameDomain Unifier;
Standard_Boolean isUnifyEdges = Standard_True;
Standard_Boolean isUnifyFaces = Standard_False; //only edges
Standard_Boolean isConcatBSplines = Standard_True;
Unifier.Initialize(aResult, isUnifyEdges, isUnifyFaces, isConcatBSplines);
Unifier.SetLinearTolerance(myTolerance);
Unifier.Build();
myShape = Unifier.Shape();
TopoDS_Shape aRes = BlockFix::FixRanges(myShape,myTolerance);
myShape = aRes;

19
src/GEOMImpl/GEOMImpl_BlockDriver.cxx
View File

@ -63,6 +63,8 @@
#include <BRepExtrema_ExtPF.hxx>
#include <BRepExtrema_DistShapeShape.hxx>
#include <ShapeUpgrade_UnifySameDomain.hxx>
#include <TopAbs.hxx>
#include <TopoDS.hxx>
#include <TopoDS_Shape.hxx>
@ -742,10 +744,19 @@ Standard_Integer GEOMImpl_BlockDriver::Execute(Handle(TFunction_Logbook)& log) c
Standard_NullObject::Raise("Null Shape given");
}
BlockFix_UnionFaces aFaceUnifier;
aFaceUnifier.GetOptimumNbFaces() = 0; // To force union faces.
aShape = aFaceUnifier.Perform(aBlockOrComp);
//BlockFix_UnionFaces aFaceUnifier;
//aFaceUnifier.GetOptimumNbFaces() = 0; // To force union faces.
//aShape = aFaceUnifier.Perform(aBlockOrComp);
// Use OCCT algo ShapeUpgrade_UnifySameDomain instead of BlockFix_UnionFaces:
Standard_Boolean isUnifyEdges = Standard_False;
Standard_Boolean isUnifyFaces = Standard_True;
Standard_Boolean isConcatBSplines = Standard_True;
ShapeUpgrade_UnifySameDomain aUnifier (aBlockOrComp,
isUnifyEdges, isUnifyFaces, isConcatBSplines);
aUnifier.SetLinearTolerance(Precision::Confusion());
aUnifier.SetAngularTolerance(Precision::Confusion());
aUnifier.Build();
aShape = aUnifier.Shape();
} else { // unknown function type
return 0;
}