diff --git a/src/ifcgeom/mapping/IfcCartesianTransformationOperator2D.cpp b/src/ifcgeom/mapping/IfcCartesianTransformationOperator2D.cpp
index 496aac2e23..70eedc7988 100644
--- a/src/ifcgeom/mapping/IfcCartesianTransformationOperator2D.cpp
+++ b/src/ifcgeom/mapping/IfcCartesianTransformationOperator2D.cpp
@@ -55,7 +55,8 @@ taxonomy::ptr mapping::map_impl(const IfcSchema::IfcCartesianTransformationOpera
 		axis3,
 		origin;
 
-	m->components().transposeInPlace();
+	// @todo why was this here
+	// m->components().transposeInPlace();
 
 	return m;
 }
diff --git a/src/ifcgeom/mapping/IfcSectionedSolidHorizontal.cpp b/src/ifcgeom/mapping/IfcSectionedSolidHorizontal.cpp
index fdb1946ff5..5dbee7cff7 100644
--- a/src/ifcgeom/mapping/IfcSectionedSolidHorizontal.cpp
+++ b/src/ifcgeom/mapping/IfcSectionedSolidHorizontal.cpp
@@ -253,8 +253,7 @@ taxonomy::ptr mapping::map_impl(const IfcSchema::IfcSectionedSolidHorizontal* in
 				// @todo should this not be initialized by default? matrix4 already has a 'lazy identity' mechanism.
 				loft->children.back()->matrix = taxonomy::make<taxonomy::matrix4>();
 			}
-			// @todo why do we do a transpose here, that doesn't seem right.
-			auto m = (m4b * loft->children.back()->matrix->ccomponents().transpose()).eval();
+			auto m = (m4b * loft->children.back()->matrix->ccomponents()).eval();
 			loft->children.back()->matrix->components() = m;
 		}
 	}