sparse: fix sorting of indices when needed

jcarpent · jcarpent · commit f728dfbff3e3 · 2025-03-26T18:31:59.000+01:00
Only available from Eigen &gt;= 3.4.90
diff --git a/include/eigenpy/sparse/eigen-from-python.hpp b/include/eigenpy/sparse/eigen-from-python.hpp
@@ -1,5 +1,5 @@
 //
-// Copyright (c) 2024 INRIA
+// Copyright (c) 2024-2025 INRIA
 //
 
 #ifndef __eigenpy_sparse_eigen_from_python_hpp__
@@ -156,6 +156,10 @@ void eigen_sparse_matrix_from_py_construct(
     }
     MapMatOrRefType sparse_map(m, n, nnz, indptr.data(), indices_ptr, data_ptr);
 
+#if EIGEN_VERSION_AT_LEAST(3, 4, 90)
+    sparse_map.sortInnerIndices();
+#endif
+
     new (raw_ptr) MatOrRefType(sparse_map);
   }
 

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`//`
`2`		`-// Copyright (c) 2024 INRIA`
	`2`	`+// Copyright (c) 2024-2025 INRIA`
`3`	`3`	`//`
`4`	`4`
`5`	`5`	`#ifndef __eigenpy_sparse_eigen_from_python_hpp__`
`@@ -156,6 +156,10 @@ void eigen_sparse_matrix_from_py_construct(`
`156`	`156`	`}`
`157`	`157`	`MapMatOrRefType sparse_map(m, n, nnz, indptr.data(), indices_ptr, data_ptr);`
`158`	`158`
	`159`	`+#if EIGEN_VERSION_AT_LEAST(3, 4, 90)`
	`160`	`+ sparse_map.sortInnerIndices();`
	`161`	`+#endif`
	`162`	`+`
`159`	`163`	`new (raw_ptr) MatOrRefType(sparse_map);`
`160`	`164`	`}`
`161`	`165`