#19976: Optimize Cython dependency checking
-------------------------------------+-------------------------------------
Reporter: jdemeyer | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-7.1
Component: packages: | Resolution:
standard | Merged in:
Keywords: | Reviewers:
Authors: Jeroen Demeyer | Work issues:
Report Upstream: Reported | Commit:
upstream. No feedback yet. | f82bc310bf2c14801d1b9a2d8af28d926dfe2c87
Branch: | Stopgaps:
u/jdemeyer/optimize_cython_dependency_checking|
Dependencies: |
-------------------------------------+-------------------------------------
Changes (by jdemeyer):
* cc: robertwb (added)
* status: new => needs_review
* commit: => f82bc310bf2c14801d1b9a2d8af28d926dfe2c87
Old description:
> Upgrade to Cython 0.23.4 and add the following patch:
> [https://github.com/cython/cython/pull/461]
>
> This improves the dependency checking in Cython. One consequence is that
> the following is completely gone when doing `./sage -b`:
> {{{
> copying /usr/local/src/sage-
> git/src/build/cythonized/sage/matroids/minorfix.h -> /usr/local/src/sage-
> git/local/lib/python2.7/site-packages/sage/matroids
> copying /usr/local/src/sage-
> git/src/build/cythonized/sage/stats/distributions/dgs_misc.h ->
> /usr/local/src/sage-git/local/lib/python2.7/site-
> packages/sage/stats/distributions
> ...
> copying /usr/local/src/sage-
> git/src/build/cythonized/sage/symbolic/pynac_cc.h -> /usr/local/src/sage-
> git/local/lib/python2.7/site-packages/sage/symbolic
> copying /usr/local/src/sage-
> git/src/build/cythonized/sage/symbolic/ginac_wrap.h -> /usr/local/src
> /sage-git/local/lib/python2.7/site-packages/sage/symbolic
> }}}
>
> '''Tarball''': http://cython.org/release/Cython-0.23.4.tar.gz
New description:
Upgrade to Cython 0.23.4 and add the following patch:
[https://github.com/cython/cython/pull/461]
This improves the dependency checking in Cython. One consequence is that
the following is completely gone when doing `./sage -b`:
{{{
copying /usr/local/src/sage-
git/src/build/cythonized/sage/matroids/minorfix.h -> /usr/local/src/sage-
git/local/lib/python2.7/site-packages/sage/matroids
copying /usr/local/src/sage-
git/src/build/cythonized/sage/stats/distributions/dgs_misc.h ->
/usr/local/src/sage-git/local/lib/python2.7/site-
packages/sage/stats/distributions
...
copying /usr/local/src/sage-
git/src/build/cythonized/sage/symbolic/pynac_cc.h -> /usr/local/src/sage-
git/local/lib/python2.7/site-packages/sage/symbolic
copying /usr/local/src/sage-
git/src/build/cythonized/sage/symbolic/ginac_wrap.h -> /usr/local/src
/sage-git/local/lib/python2.7/site-packages/sage/symbolic
}}}
'''Tarball''': http://cython.org/release/Cython-0.23.4.tar.gz
------------------
Timings for `./sage -b` when no changes need to be made (best out of 6):
'''Before''':
{{{
real 0m4.402s
user 0m3.502s
sys 0m0.523s
}}}
'''After''':
{{{
real 0m3.478s
user 0m3.061s
sys 0m0.320s
}}}
The difference for the Cythonization part of `./sage -b` (best out of 6):
'''Before''':
{{{
Finished Cythonizing, time: 3.63 seconds.
}}}
'''After''':
{{{
Finished Cythonizing, time: 2.71 seconds.
}}}
--
Comment:
New commits:
||[http://git.sagemath.org/sage.git/commit/?id=f82bc310bf2c14801d1b9a2d8af28d926dfe2c87
f82bc31]||{{{Optimize Cython dependency checking}}}||
--
Ticket URL: <http://trac.sagemath.org/ticket/19976#comment:5>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
