Glassvision/IfcOpenShell
1
0
Fork 0
mirror of https://github.com/IfcOpenShell/IfcOpenShell.git synced 2026-09-21 14:23:53 +00:00
Code Issues Packages Projects Releases Wiki Activity

Check for orientation of wires when creating outer boundary of face

Browse Source
This commit is contained in:
Thomas Krijnen
2015-05-21 13:01:01 +00:00
parent 771f78a671
commit 9b64c6f4b9
1 changed files with 17 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ifcgeom/IfcGeomFaces.cpp
+17 -3
View File
@@ -85,6 +85,7 @@
#include <Standard_Failure.hxx> #include <Standard_Failure.hxx>
#include <BRep_Tool.hxx> #include <BRep_Tool.hxx>
#include <BRepCheck_Face.hxx>
#include <BRepBuilderAPI_Transform.hxx> #include <BRepBuilderAPI_Transform.hxx>
#ifdef USE_IFC4 #ifdef USE_IFC4
@@ -92,8 +93,6 @@
#include <TColgp_Array2OfPnt.hxx> #include <TColgp_Array2OfPnt.hxx>
#include <TColStd_Array1OfReal.hxx> #include <TColStd_Array1OfReal.hxx>
#include <TColStd_Array1OfInteger.hxx> #include <TColStd_Array1OfInteger.hxx>
#include <BRepCheck_Face.hxx>
#endif #endif
#include "../ifcgeom/IfcGeom.h" #include "../ifcgeom/IfcGeom.h"
@@ -155,7 +154,7 @@ bool IfcGeom::Kernel::convert(const IfcSchema::IfcFace* l, TopoDS_Shape& face) {
IfcSchema::IfcFaceBound* bound = *it; IfcSchema::IfcFaceBound* bound = *it;
IfcSchema::IfcLoop* loop = bound->Bound(); IfcSchema::IfcLoop* loop = bound->Bound();
const bool same_sense = bound->Orientation(); bool same_sense = bound->Orientation();
const bool is_interior = const bool is_interior =
!bound->is(IfcSchema::Type::IfcFaceOuterBound) && !bound->is(IfcSchema::Type::IfcFaceOuterBound) &&
(num_bounds > 1) && (num_bounds > 1) &&
@@ -216,6 +215,21 @@ bool IfcGeom::Kernel::convert(const IfcSchema::IfcFace* l, TopoDS_Shape& face) {
if (mf->IsDone()) { if (mf->IsDone()) {
TopoDS_Face outer_face_bound = mf->Face(); TopoDS_Face outer_face_bound = mf->Face();
if (BRepCheck_Face(outer_face_bound).OrientationOfWires() == BRepCheck_BadOrientationOfSubshape) {
wire.Reverse();
same_sense = !same_sense;
delete mf;
if (face_surface.IsNull()) {
mf = new BRepBuilderAPI_MakeFace(wire);
} else {
mf = new BRepBuilderAPI_MakeFace(face_surface, wire);
}
ShapeFix_Face fix(mf->Face());
fix.FixOrientation();
fix.Perform();
outer_face_bound = fix.Face();
}
// If the wires are reversed the face needs to be reversed as well in order // If the wires are reversed the face needs to be reversed as well in order
// to maintain the counter-clock-wise ordering of the bounding wire's vertices. // to maintain the counter-clock-wise ordering of the bounding wire's vertices.
bool all_reversed = true; bool all_reversed = true;