Set rows and cols in exposeMatrixQ with def and not prop

Author: Lucas-Haubert

include/nanoeigenpy/decompositions/sparse/sparse-qr.hpp

@@ -43,8 +43,8 @@ void exposeMatrixQ(nb::module_ m) {
     nb::class_<QType>(m, "SparseQRMatrixQReturnType")
         .def(nb::init<const SparseQRType&>(), "qr"_a)
 
-        .def_prop_ro("rows", &QType::rows)
-        .def_prop_ro("cols", &QType::cols)
+        .def("rows", &QType::rows)
+        .def("cols", &QType::cols)
 
         .def(
             "__matmul__",

tests/test_sparse_qr.py

@@ -40,8 +40,8 @@
 R = spqr.matrixR()
 P = spqr.colsPermutation()
 
-assert spqr.matrixQ().rows == dim
-assert spqr.matrixQ().cols == dim
+assert spqr.matrixQ().rows() == dim
+assert spqr.matrixQ().cols() == dim
 assert R.shape[0] == dim
 assert R.shape[1] == dim
 assert P.indices().size == dim