From 2f485ec9aceb72330854b3cb5c206621d2d5bc1b Mon Sep 17 00:00:00 2001 From: Richard Brice <37087370+RickBrice@users.noreply.github.com> Date: Fri, 10 Jul 2026 07:55:51 -0700 Subject: [PATCH] Fixes double unit conversion when convert-back-units are used --- src/ifcwrap/IfcGeomWrapper.i | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/ifcwrap/IfcGeomWrapper.i b/src/ifcwrap/IfcGeomWrapper.i index 81c2a53795..404ecd7ded 100644 --- a/src/ifcwrap/IfcGeomWrapper.i +++ b/src/ifcwrap/IfcGeomWrapper.i @@ -977,12 +977,14 @@ struct ShapeRTTI : public boost::static_visitor if (item == nullptr) { throw IfcParse::IfcException("Failed to convert placement"); } + /* if (st.get().get()) { // we pass the settings to the Transformation object, but access the data just offloads to the // generic cartesian_base so there's no time to apply the settings to the translation part. item = ifcopenshell::geometry::taxonomy::matrix4::ptr(item->clone_()); item->components().col(3).head<3>() /= kernel.settings().get().get(); } + */ return new IfcGeom::Transformation(kernel.settings(), item); } else { if (!representation) {