From 9f44bbb57b9e457131f9430924df1bd0eb0c991c Mon Sep 17 00:00:00 2001 From: Justin Carpentier <justin.carpentier@inria.fr> Date: Tue, 18 Aug 2020 11:21:41 +0200 Subject: [PATCH] core: remove useless transpose() --- include/eigenpy/eigen-allocator.hpp | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/include/eigenpy/eigen-allocator.hpp b/include/eigenpy/eigen-allocator.hpp index dc6a84c..49fbadd 100644 --- a/include/eigenpy/eigen-allocator.hpp +++ b/include/eigenpy/eigen-allocator.hpp @@ -75,10 +75,7 @@ namespace eigenpy const Eigen::MatrixBase<MatrixOut> & dest) { MatrixOut & dest_ = const_cast<MatrixOut &>(dest.derived()); - if(dest.rows() == input.rows()) - dest_ = input.template cast<NewScalar>(); - else - dest_ = input.transpose().template cast<NewScalar>(); + dest_ = input.template cast<NewScalar>(); } }; @@ -168,11 +165,7 @@ namespace eigenpy if(pyArray_type_code == Scalar_type_code) // no cast needed { MapType map_pyArray = NumpyMap<MatType,Scalar>::map(pyArray); - if(mat.rows() == map_pyArray.rows()) - map_pyArray = mat; - else - map_pyArray = mat.transpose(); - return; + map_pyArray = mat; } switch(pyArray_type_code) -- GitLab