core: fix issue with Vector types

jcarpent · jcarpent · commit 7d28bd5e914f · 2022-12-07T19:26:03.000+01:00
a vector should be either C or F continuous
diff --git a/include/eigenpy/eigen-allocator.hpp b/include/eigenpy/eigen-allocator.hpp
@@ -245,7 +245,8 @@ inline bool is_arr_layout_compatible_with_mat_type(PyArrayObject *pyArray) {
   bool is_array_F_cont = PyArray_IS_F_CONTIGUOUS(pyArray);
   return (MatType::IsRowMajor && is_array_C_cont) ||
          (!MatType::IsRowMajor && is_array_F_cont) ||
-         MatType::IsVectorAtCompileTime;
+         (MatType::IsVectorAtCompileTime &&
+          (is_array_C_cont || is_array_F_cont));
 }
 
 template <typename MatType, int Options, typename Stride>

Original file line number	Diff line number	Diff line change
`@@ -245,7 +245,8 @@ inline bool is_arr_layout_compatible_with_mat_type(PyArrayObject *pyArray) {`
`245`	`245`	`bool is_array_F_cont = PyArray_IS_F_CONTIGUOUS(pyArray);`
`246`	`246`	`return (MatType::IsRowMajor && is_array_C_cont) \|\|`
`247`	`247`	`(!MatType::IsRowMajor && is_array_F_cont) \|\|`
`248`		`- MatType::IsVectorAtCompileTime;`
	`248`	`+ (MatType::IsVectorAtCompileTime &&`
	`249`	`+ (is_array_C_cont \|\| is_array_F_cont));`
`249`	`250`	`}`
`250`	`251`
`251`	`252`	`template <typename MatType, int Options, typename Stride>`