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

Don't apply too many half space subtraction operands at once

Browse Source
This commit is contained in:
Thomas Krijnen
2019-10-15 15:06:51 +02:00
parent 503029733c
commit 4000fef4c7
1 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ifcgeom/IfcGeomShapes.cpp
+9 -1
View File
@@ -501,9 +501,17 @@ bool IfcGeom::Kernel::convert(const IfcSchema::IfcBooleanResult* l, TopoDS_Shape
second_operands.push_back(operand2); second_operands.push_back(operand2);
if (occ_op == BOPAlgo_CUT) { if (occ_op == BOPAlgo_CUT) {
bool process_as_list = true; int n_half_space_operands = 0;
bool process_as_list = false;
while (true) { while (true) {
auto res1 = operand1->as<IfcSchema::IfcBooleanResult>(); auto res1 = operand1->as<IfcSchema::IfcBooleanResult>();
if (res1 && res1->SecondOperand()->as<IfcSchema::IfcHalfSpaceSolid>() && ++n_half_space_operands > 8) {
// There is something peculiar about many half space subtraction operands that OCCT does not like.
// Often these are used to create a semi-curved arch, as is the case in 693. Supplying all these
// operands at once apparently leads to too many edge-edge interference checks.
process_as_list = false;
break;
}
if (res1) { if (res1) {
if (res1->Operator() == op) { if (res1->Operator() == op) {
operand1 = res1->FirstOperand(); operand1 = res1->FirstOperand();