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

Fix --convert-back-units on transformation object #8137

Browse Source
This commit is contained in:
Thomas Krijnen
2026-06-04 22:15:35 +02:00
parent 94fab271cd
commit 1f2b20fd86
1 changed files with 17 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ifcgeom/IfcGeomElement.h
+17 -5
View File
@@ -35,13 +35,25 @@ namespace IfcGeom {
class Transformation { class Transformation {
private: private:
ifcopenshell::geometry::Settings settings_; ifcopenshell::geometry::Settings settings_;
ifcopenshell::geometry::taxonomy::matrix4::ptr matrix_; ifcopenshell::geometry::taxonomy::matrix4::ptr matrix_, matrix_orig_units_;
public: public:
Transformation(const ifcopenshell::geometry::Settings& settings, const ifcopenshell::geometry::taxonomy::matrix4::ptr& matrix) Transformation(const ifcopenshell::geometry::Settings& settings, const ifcopenshell::geometry::taxonomy::matrix4::ptr& matrix)
: settings_(settings) : settings_(settings), matrix_(matrix)
, matrix_(matrix) {
{} const bool convert = settings.get<ifcopenshell::geometry::settings::ConvertBackUnits>().get();
auto unit_magnitude = settings.get<ifcopenshell::geometry::settings::LengthUnit>().get();
if (matrix_ && convert && unit_magnitude != 1.0) {
matrix_orig_units_ = ifcopenshell::geometry::taxonomy::make<ifcopenshell::geometry::taxonomy::matrix4>(*matrix);
// only multiple the translation components of the matrix with the unit magnitude, not the rotation/scaling components
matrix_orig_units_->components().col(3).head<3>() /= unit_magnitude;
} else {
matrix_orig_units_ = nullptr;
}
}
const ifcopenshell::geometry::taxonomy::matrix4::ptr& data() const { const ifcopenshell::geometry::taxonomy::matrix4::ptr& data() const {
if (matrix_orig_units_) {
return matrix_orig_units_;
}
if (matrix_) { if (matrix_) {
return matrix_; return matrix_;
} }