From c8d39cc4815d0d3d83fe33f60dbb8478697da12e Mon Sep 17 00:00:00 2001 From: Thomas Krijnen Date: Thu, 21 May 2026 14:17:17 +0200 Subject: [PATCH] At least make sure that hybrid-cgal-simple-opencascade processes the elements correctly #8052 --- src/ifcgeom/kernels/cgal/CgalKernel.cpp | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/ifcgeom/kernels/cgal/CgalKernel.cpp b/src/ifcgeom/kernels/cgal/CgalKernel.cpp index ba46adcd12..6e8d957188 100644 --- a/src/ifcgeom/kernels/cgal/CgalKernel.cpp +++ b/src/ifcgeom/kernels/cgal/CgalKernel.cpp @@ -214,8 +214,13 @@ bool CgalKernel::convert(const taxonomy::shell::ptr l, cgal_shape_t& shape) { } catch (...) {} if (!success) { - logger::message(logger::LOG_WARNING, "Failed to convert face:", f->instance); - continue; + if (this->partial_success_is_success) { + logger::message(logger::LOG_WARNING, "Failed to convert face, skipping:", f->instance); + continue; + } else { + logger::message(logger::LOG_ERROR, "Failed to convert face:", f->instance); + return false; + } } // std::cout << "Face in ConnectedFaceSet: " << std::endl;