Minor Improvement to Sympification of Tuples and Lists

Author: ShikharJ

symengine/lib/symengine_wrapper.pyx

@@ -460,6 +460,16 @@ def sympify(a):
     """
     if isinstance(a, str):
         return c2py(symengine.parse(a.encode("utf-8")))
+    elif isinstance(a, tuple):
+        v = []
+        for e in a:
+            v.append(sympify(e))
+        return tuple(v)
+    elif isinstance(a, list):
+        v = []
+        for e in a:
+            v.append(sympify(e))
+        return v
     return _sympify(a, True)
 
 
@@ -494,16 +504,6 @@ def _sympify(a, raise_error=True):
         return RealDouble(a)
     elif isinstance(a, complex):
         return ComplexDouble(a)
-    elif isinstance(a, tuple):
-        v = []
-        for e in a:
-            v.append(_sympify(e, True))
-        return tuple(v)
-    elif isinstance(a, list):
-        v = []
-        for e in a:
-            v.append(_sympify(e, True))
-        return v
     elif hasattr(a, '_symengine_'):
         return _sympify(a._symengine_(), raise_error)
     elif hasattr(a, '_sympy_'):

symengine/tests/test_sympify.py

@@ -9,6 +9,8 @@ def test_sympify1():
     assert sympify(2) != Integer(1)
     assert sympify(-5) == Integer(-5)
     assert sympify(Integer(3)) == Integer(3)
+    assert sympify(('0', '0')) == (0, 0)
+    assert sympify(['0', '0']) == [0, 0]
     assert sympify("3+5") == Integer(8)
     assert true == sympify(True)
     assert false == sympify(False)