Mon, 12/09/2019 - 12:43
Forums:
Hello.
I am trying to fuse face and face by following codes, but it is not working.
I use OpenCascade version 7.4.0.
BRepBuilderAPI_MakeWire makeCheckWire;
makeCheckWire.Add( BRepBuilderAPI_MakeEdge( gp_Pnt( 0.0, 0.0, 0.0 ), gp_Pnt( 0.0, 0.0, 100.0 ) ) );
makeCheckWire.Add( BRepBuilderAPI_MakeEdge( gp_Pnt( 0.0, 0.0, 100.0 ), gp_Pnt( 100.0, 0.0, 100.0 ) ) );
makeCheckWire.Add( BRepBuilderAPI_MakeEdge( gp_Pnt( 100.0, 0.0, 100.0 ), gp_Pnt( 100.0, 0.0, 0.0 ) ) );
makeCheckWire.Add( BRepBuilderAPI_MakeEdge( gp_Pnt( 100.0, 0.0, 0.0 ), gp_Pnt( 0.0, 0.0, 0.0 ) ) );
BRepBuilderAPI_MakeFace makerFace1( makeCheckWire.Wire() );
TopoDS_Face face1 = makerFace1.Face();
BRepBuilderAPI_MakeWire makeCheckWire2;
double dAdjustX = 40.0;
makeCheckWire2.Add( BRepBuilderAPI_MakeEdge( gp_Pnt( 0.0, 0.0, 0.0 ), gp_Pnt( 0.0, 0.0, -20.0 ) ) );
makeCheckWire2.Add( BRepBuilderAPI_MakeEdge( gp_Pnt( 0.0, 0.0, -20.0 ), gp_Pnt( dAdjustX, 0.0, -20.0 ) ) );
makeCheckWire2.Add( BRepBuilderAPI_MakeEdge( gp_Pnt( dAdjustX, 0.0, -20.0 ), gp_Pnt( dAdjustX, 0.0, 0.0 ) ) );
makeCheckWire2.Add( BRepBuilderAPI_MakeEdge( gp_Pnt( dAdjustX, 0.0, 0.0 ), gp_Pnt( 0.0, 0.0, 0.0 ) ) );
BRepBuilderAPI_MakeFace makerFace2( makeCheckWire2.Wire() );
TopoDS_Face face2 = makerFace2.Face();
BRepAlgoAPI_Fuse fuseShape( face1, face2 );
fuseShape.Build();
TopoDS_Shape shape = fuseShape.Shape();
ShapeUpgrade_UnifySameDomain unif( shape, Standard_True, Standard_True, Standard_False );
unif.Build();
shape = unif.Shape();
When I use OpenCascade version 6.8.0, 2 faces are fused by ShapeUpgrade_UnifySameDomain class.
But After I upgraded OpenCascade version from 6.8.0 to 7.4.0, These 2 faces are not fused.
Do you know how to solve this problem?
Best regards.
Attachments:
Mon, 12/09/2019 - 13:10
Currently, your faces have different normal directions. Reversing one of your faces will help.
Tue, 12/10/2019 - 04:36
Eugeny, thank you for the Information !
I fixed my code like below and It works good.
TopoDS_Face face2 = makerFace2.Face();
↓
TopoDS_Face face2 = makerFace2.Face();
face2 .Reverse();
I have one more question.
In the case of not plane face, how can I know if I should use reverse or not ?
Tue, 12/10/2019 - 08:12
Hello Keito,
You can create a shell from the compound and use the ShapeFix_Shell to fix the face orientation in the shell. Then you can use the UnifySameDom on the fixed shell.
Best regards, Eugeny.
Wed, 12/11/2019 - 05:16
Hello Eugeny,
Thank you for your reply.
I tried to make shell and fix this, but I could not connect cylinder face.
Is something wrong? I used following codes.
TopExp_Explorer expFace;
BRepBuilderAPI_MakeWire makeCheckWire;
TopoDS_Edge edgeArc1 = BRepBuilderAPI_MakeEdge(
GC_MakeArcOfCircle( gp_Circ( gp_Ax2(gp_Pnt(0.0,0.0,0.0), gp_Dir(1.0,0.0,0.0)), 5.0 ),
gp_Pnt( 0.0, -5.0, 0.0 ),
gp_Pnt( 0.0, 5.0, 0.0 ),
Standard_False ).Value() );
makeCheckWire.Add( edgeArc1 );
makeCheckWire.Add( BRepBuilderAPI_MakeEdge( gp_Pnt( 0.0, -5.0, 0.0 ), gp_Pnt( 0.0, 5.0, 0.0 ) ) );
BRepBuilderAPI_MakeFace makerFace1( makeCheckWire.Wire() );
TopoDS_Face face1 = makerFace1.Face();
TopoDS_Shape shape1 = BRepPrimAPI_MakePrism( face1, gp_Vec( 20.0, 0.0, 0.0 ) ).Shape();
expFace.Init( shape1, TopAbs_FACE );
for( ; expFace.More(); expFace.Next() ){
TopoDS_Face face = TopoDS::Face( expFace.Current() );
BRepAdaptor_Surface adapetrSurface( face );
if( adapetrSurface.GetType() == GeomAbs_Cylinder ){
face1 = face;
break;
}
}
BRepBuilderAPI_MakeWire makeCheckWire2;
TopoDS_Edge edgeArc2 = BRepBuilderAPI_MakeEdge(
GC_MakeArcOfCircle( gp_Circ( gp_Ax2(gp_Pnt(0.0,0.0,0.0), gp_Dir(1.0,0.0,0.0)), 5.0 ),
gp_Pnt( 0.0, 5.0, 0.0 ),
gp_Pnt( 0.0, -5.0, 0.0 ),
Standard_False ).Value() );
makeCheckWire2.Add( edgeArc2 );
makeCheckWire2.Add( BRepBuilderAPI_MakeEdge( gp_Pnt( 0.0, -5.0, 0.0 ), gp_Pnt( 0.0, 5.0, 0.0 ) ) );
BRepBuilderAPI_MakeFace makerFace2( makeCheckWire2.Wire() );
TopoDS_Face face2 = makerFace2.Face();
TopoDS_Shape shape2 = BRepPrimAPI_MakePrism( face2, gp_Vec( 20.0, 0.0, 0.0 ) ).Shape();
expFace.Init( shape2, TopAbs_FACE );
for( ; expFace.More(); expFace.Next() ){
TopoDS_Face face = TopoDS::Face( expFace.Current() );
BRepAdaptor_Surface adapetrSurface( face );
if( adapetrSurface.GetType() == GeomAbs_Cylinder ){
face2 = face;
break;
}
}
face2.Reverse(); // for test, reverse face.
BRepAlgoAPI_Fuse fuseShape( face1, face2 );
fuseShape.Build();
TopoDS_Shell shell;
BRep_Builder builder;
builder.MakeShell( shell );
TopExp_Explorer exFace( fuseShape.Shape(), TopAbs_FACE );
for( ; exFace.More(); exFace.Next() ){
builder.Add( shell, exFace.Current() );
}
ShapeFix_Shell fixShell;
fixShell.FixFaceOrientation( shell, Standard_True, Standard_False );
ShapeUpgrade_UnifySameDomain unif( shell, Standard_True, Standard_True, Standard_False );
unif.Build();
TopoDS_Shape shape = unif.Shape(); // cylinder faces are divied.
Best regards, Keito.
Wed, 12/11/2019 - 09:10
Hello Keito,
Could you please dump the shapes after fuse? You can use either BinTools::Write or BRepTools::Write for that.
One remark for your code - you can use arc edge directly for making the cylindrical face, instead of making the planar face first, making solid and looking for the cylindrical face in that solid.
Here is the draw script which produces the desired result:
Wed, 12/11/2019 - 10:46
Hello Eugeny,
Thank you for the information and coding advice.
I fixed my code like below, but result is same.
Is something wrong?
~~~~
fuseShape.Build();
TopoDS_Shape shapeTmp = fuseShape.Shape();
BinTools::Write( shapeTmp, "@test" );
BinTools::Read( shapeTmp, "@test" );
TopoDS_Shell shell;
BRep_Builder builder;
builder.MakeShell( shell );
TopExp_Explorer exFace( shapeTmp , TopAbs_FACE );
for( ; exFace.More(); exFace.Next() ){
builder.Add( shell, exFace.Current() );
}
ShapeFix_Shell fixShell;
fixShell.FixFaceOrientation( shell, Standard_True, Standard_False );
ShapeUpgrade_UnifySameDomain unif( shell, Standard_True, Standard_True, Standard_False );
unif.Build();
TopoDS_Shape shape = unif.Shape(); // cylinder faces are divied.
Best regards, Keito.
Wed, 12/11/2019 - 11:22
I suggested to dump the shapes so you could load it in draw and try to reproduce the issue with unification there. No need to dump and restore it in your code.
Anyway, this looks like an issue in OCCT, so you could register a bug in OCCT mantis bugtracker and contact OCCT Support via the Contact form to advance the issue.
Thu, 12/12/2019 - 07:45
Thany you for the imformation.
I will register a bug in OCCT mantis bugtracker.