Merge pull request #191 from isuruf/atoms

Fix DenseMatrix.atoms

Author: ShikharJ

symengine/lib/symengine_wrapper.pyx

@@ -3581,7 +3581,7 @@ cdef class DenseMatrixBase(MatrixBase):
             s = set()
             if (isinstance(self, types)):
                 s.add(self)
-            for arg in self.tolist():
+            for arg in self:
                 s.update(arg.atoms(*types))
             return s
         else:

symengine/tests/test_matrices.py

@@ -520,3 +520,9 @@ def test_immutablematrix():
     Z = X.col_join(Y)
     assert isinstance(Z, ImmutableMatrix)
     assert Z == ImmutableMatrix([[1, 2], [3, 4], [5, 6]])
+
+def test_atoms():
+    a = Symbol("a")
+    b = Symbol("b")
+    X = DenseMatrix([[a, 2], [b, 4]])
+    assert X.atoms(Symbol) == set([a, b])