Glassvision/IfcOpenShell
1
0
Fork 0
mirror of https://github.com/IfcOpenShell/IfcOpenShell.git synced 2026-09-22 13:58:12 +00:00
Code Issues Packages Projects Releases Wiki Activity

Don't raise error on eliminated compcurve polygon points

Browse Source
This commit is contained in:
Thomas Krijnen
2020-01-12 15:18:12 +01:00
parent 98d8a3477d
commit 68763f4988
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ifcgeom/IfcGeomWires.cpp
+5 -1
View File
@@ -387,7 +387,8 @@ bool IfcGeom::Kernel::convert(const IfcSchema::IfcCompositeCurve* l, TopoDS_Wire
TopoDS_Wire segment; TopoDS_Wire segment;
if (!convert_wire(curve, segment)) { if (!convert_wire(curve, segment)) {
Logger::Message(Logger::LOG_ERROR, "Failed to convert curve:", curve); const bool failed_on_purpose = curve->as<IfcSchema::IfcPolyline>() && !segment.IsNull();
Logger::Message(failed_on_purpose ? Logger::LOG_WARNING : Logger::LOG_ERROR, "Failed to convert curve:", curve);
continue; continue;
} }
@@ -625,6 +626,9 @@ bool IfcGeom::Kernel::convert(const IfcSchema::IfcPolyline* l, TopoDS_Wire& resu
remove_duplicate_points_from_loop(polygon, closed_by_proximity, eps); remove_duplicate_points_from_loop(polygon, closed_by_proximity, eps);
if (polygon.Length() < 2) { if (polygon.Length() < 2) {
// We somehow need to signal we fail this curve on purpose not to trigger an error.
BRep_Builder B;
B.MakeWire(result);
return false; return false;
} }