Merge pull request #350 from rikardn/fix347

Properly handle sympy matrix for Matrix constructor

Author: isuruf

symengine/lib/symengine_wrapper.pyx

@@ -3170,7 +3170,7 @@ cdef class DenseMatrixBase(MatrixBase):
             self.thisptr = new symengine.DenseMatrix(row, col)
             return
         if col is None:
-            v = row
+            v = sympify(row)
             row = 0
         cdef symengine.vec_basic v_
         cdef DenseMatrixBase A

symengine/tests/test_sympy_conv.py

@@ -783,3 +783,12 @@ def test_pynumber():
 
     b = b / x
     assert isinstance(b, PyNumber)
+
+
+@unittest.skipIf(not have_sympy, "SymPy not installed")
+def test_construct_dense_matrix():
+    # Test for issue #347
+    A = sympy.Matrix([[1, 2], [3, 5]])
+    B = DenseMatrix(A)
+    assert B.shape == (2, 2)
+    assert list(B) == [1, 2, 3, 5]