Upgrade to SymPy-1.0

[rwst]

https://github.com/sympy/sympy/wiki/Release-Notes-for-1.0

It could be installed as usually
https://github.com/sympy/sympy/releases/download/sympy-1.0/sympy-1.0.tar.gz
or via pip install -U sympy.

Component: packages: standard

Author: Ralf Stephan

Branch/Commit: 3872f81

Reviewer: Travis Scrimshaw, Ralf Stephan, Volker Braun

Issue created by migration from https://trac.sagemath.org/ticket/20185

[rwst]

Description changed:

--- 
+++ 
@@ -1,3 +1,5 @@
 https://github.com/sympy/sympy/wiki/Release-Notes-for-1.0
 
-It could be installed as usually or via `pip install -U sympy`.
+It could be installed as usually 
+https://github.com/sympy/sympy/releases/download/sympy-1.0/sympy-1.0.tar.gz
+or via `pip install -U sympy`.

[rwst]

Branch: u/rws/upgrade_to_sympy_1_0

[rwst]

Commit: 6971607

[rwst]

comment:3

$ ./sage -p mpmath
Successfully installed mpmath-0.19
$ ./sage -p sympy
building sympy
Please install the mpmath package with a version >= 0.19
Error building sympy

Any hints?

---

New commits:

6971607

pkg version/chksum/patch removed

[kiwifb]

comment:4

Replying to @rwst:

$ ./sage -p mpmath
Successfully installed mpmath-0.19
$ ./sage -p sympy
building sympy
Please install the mpmath package with a version >= 0.19
Error building sympy

Any hints?

---

New commits:

6971607

pkg version/chksum/patch removed

I think I have. In the past (prior to this version) sympy's setup.py was importing sympy itself. The danger then was to import the previously installed sympy rather the new sympy. So we add this

export PYTHONPATH="."
echo "building sympy"
python -S setup.py build

The -S is the cause of your trouble, look it up. So you should remove the PYTHONPATH stuff, the -S and remove the corresponding comments in spkg-install.

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

9cdde13	20185: remove python -S stuff
907bedc	Merge branch 'develop' into t/20185/upgrade_to_sympy_1_0

[sagetrac-git]

Changed commit from 6971607 to 907bedc

[rwst]

comment:6

Thanks, that was it. So there are changes that affect doctests:

sage -t --warn-long 27.4 src/sage/symbolic/integration/integral.py  # 1 doctest failed
sage -t --warn-long 27.4 src/sage/symbolic/expression.pyx  # 3 doctests failed
sage -t --warn-long 27.2 src/sage/tests/french_book/recequadiff.py  # 2 doctests failed

That's only from a partial quick scan.

[rwst]

comment:7

sage -t --warn-long 27.2 src/sage/tests/french_book/recequadiff.py  # 2 doctests failed

This can be minimized to

sage: import sympy
sage: u = sympy.Function('u'); n = sympy.Symbol('n', integer=True)
sage: sympy.sympify(3*u(n), evaluate=False)
NotImplementedError: SymPy function 'u' doesn't exist

which disappears when 3*u(n) is changed to u(n)*3 or u(n)

EDIT: I will just change the order in the muls for the doctest, and open a ticket. The case is rare and can be fixed after the upgrade.

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

f92f927

fix doctests

[sagetrac-git]

Changed commit from 907bedc to f92f927