OpenCascade/occt
1
0
Fork 0
mirror of https://git.dev.opencascade.org/repos/occt.git synced 2025-04-05 18:16:23 +03:00
Code Packages Releases Activity

0022586: ShapeFix_FixSmallFace mistakenly removes a non-small pipe-like face with a seam edge

Browse Source
This commit is contained in:
MSV 2011-09-16 13:14:25 +00:00 committed by bugmaster
parent 9558a87653
commit 62110ee777
2 changed files with 58 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

54
src/QAOCC/QAOCC.cxx
View File

@ -5122,6 +5122,58 @@ Standard_Integer OCC17424 (Draw_Interpretor& di, Standard_Integer argc, const ch
return 0;
}
Standard_Integer OCC22301 (Draw_Interpretor& di, Standard_Integer argc, const char ** argv)
{
if (argc != 1) {
di << "Usage : " << argv[0] << "\n";
return 1;
}
// Create mask 1111: extent == 4
TColStd_PackedMapOfInteger aFullMask;
for (Standard_Integer i = 0; i < 4; i++)
aFullMask.Add(i);
// Create mask 1100: extent == 2
TColStd_PackedMapOfInteger aPartMask;
for (Standard_Integer i = 0; i < 2; i++)
aPartMask.Add(i);
di << "aFullMask = 1111" << "\n";
di << "aPartMask = 1100" << "\n";
Standard_Boolean isAffected;
isAffected = aFullMask.Intersect(aPartMask); // true; extent == 2 (OK)
di << "First time: aFullMask.Intersect(aPartMask), isAffected = " << (Standard_Integer)isAffected << "\n";
isAffected = aFullMask.Intersect(aPartMask); // true; extent == 0 (?)
di << "Second time: aFullMask.Intersect(aPartMask), isAffected = " << (Standard_Integer)isAffected << "\n";
isAffected = aFullMask.Subtract(aPartMask); // false (?)
di << "After two intersections: aFullMask.Subtract(aPartMask), isAffected = " << (Standard_Integer)isAffected << "\n";
return 0;
}
#include <ShapeFix_FixSmallFace.hxx>
Standard_Integer OCC22586 (Draw_Interpretor& di, Standard_Integer argc, const char ** argv)
{
if (argc != 3) {
di << "Usage : " << argv[0] << " shape resshape\n";
return 1;
}
// try to read a shape:
TopoDS_Shape aShape=DBRep::Get(argv[1]);
ShapeFix_FixSmallFace aFixSmallFaces;
aFixSmallFaces.Init (aShape);
aFixSmallFaces.Perform();
TopoDS_Shape aResShape = aFixSmallFaces.Shape();
DBRep::Set(argv[2],aResShape);
return 0;
}
void QAOCC::Commands(Draw_Interpretor& theCommands) {
const char *group = "QAOCC";
@ -5225,5 +5277,7 @@ void QAOCC::Commands(Draw_Interpretor& theCommands) {
theCommands.Add("OCC20766", "OCC20766 plane a b c d", __FILE__, OCC20766, group);
theCommands.Add("OCC20627", "OCC20627", __FILE__, OCC20627, group);
theCommands.Add("OCC17424", "OCC17424 shape X_Pnt Y_Pnt Z_Pnt X_Dir Y_Dir Z_Dir PInf", __FILE__, OCC17424, group);
theCommands.Add("OCC22301", "OCC22301", __FILE__, OCC22301, group);
theCommands.Add("OCC22586", "OCC22586 shape resshape", __FILE__, OCC22586, group);
return;
}

4
src/ShapeAnalysis/ShapeAnalysis_CheckSmallFace.cxx
View File

@ -328,6 +328,8 @@ static Standard_Boolean MinMaxSmall
Standard_Integer nb = 0;
for (TopExp_Explorer ex(F,TopAbs_EDGE); ex.More(); ex.Next()) {
TopoDS_Edge E = TopoDS::Edge (ex.Current());
if (nb == 1 && E.IsSame(E1))
continue; // ignore seam edge
TopoDS_Vertex V1,V2;
TopExp::Vertices (E,V1,V2);
gp_Pnt p1,p2;
@ -393,6 +395,8 @@ static Standard_Boolean MinMaxSmall
nb = 0;
for (TopExp_Explorer ite (F,TopAbs_EDGE); ite.More(); ite.Next()) {
TopoDS_Edge E = TopoDS::Edge (ite.Current());
if (nb == 1 && E.IsSame(E1))
continue; // ignore seam edge
TopoDS_Vertex VA,VB;
TopExp::Vertices (E,VA,VB);
if (tol < 0) {