Fix the Python binding to ScaledOrthoProjectionParameters

patrikhuber · May 20, 2018 · 5863f3b · 5863f3b
1 parent 3ad3efc
commit 5863f3b
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/python/generate-python-bindings.cpp b/python/generate-python-bindings.cpp
@@ -228,8 +228,8 @@ PYBIND11_MODULE(eos, eos_module)
         .def_property_readonly("R",
              [](const fitting::ScaledOrthoProjectionParameters& p) {
             Eigen::Matrix3f R; // we could probably use Eigen::Map
-            for (int col = 0; col < 4; ++col)
-                for (int row = 0; row < 4; ++row)
+            for (int col = 0; col < 3; ++col)
+                for (int row = 0; row < 3; ++row)
                     R(row, col) = p.R[col][row];
             return R;
     }, "Rotation matrix") // we can easily make this writable if ever required, just need to add a lambda function with the Eigen to glm matrix conversion.