Opened 11 years ago
Last modified 11 years ago
#11761 closed enhancement
Upgrade Cython to 0.15.1 — at Version 18
Reported by: | Robert Bradshaw | Owned by: | tbd |
---|---|---|---|
Priority: | major | Milestone: | sage-4.8 |
Component: | packages: standard | Keywords: | |
Cc: | François Bissey, Jean-Pierre Flori | Merged in: | |
Authors: | Robert Bradshaw | Reviewers: | Jeroen Demeyer |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
Lots of improvements and bug fixes, including generators.
See http://wiki.cython.org/ReleaseNotes-0.15.1 for details.
New spkg: http://sage.math.washington.edu/home/robertwb/cython/spkg/cython-0.15.1.spkg
Apply 11761-cython-0.15-rebased.patch and 11761_review.patch to the Sage library.
Change History (21)
comment:1 Changed 11 years ago by
Changed 11 years ago by
Attachment: | 11761-cython-0.15.patch added |
---|
comment:2 Changed 11 years ago by
Status: | new → needs_review |
---|
comment:3 Changed 11 years ago by
Cc: | François Bissey added |
---|
comment:4 Changed 11 years ago by
Cc: | Jean-Pierre Flori added |
---|
comment:5 Changed 11 years ago by
Note that 0.15.1 is around the corner, but it's not as big of a jump.
comment:6 Changed 11 years ago by
Authors: | → Robert Bradshaw |
---|---|
Description: | modified (diff) |
Would 0.15.1 require additional changes to Sage?
comment:8 follow-up: 9 Changed 11 years ago by
Replying to jason:
Just FYI, 0.15.1 is now released.
Oh, did you tell Robert?
(Although he said "it's not as big of a jump".)
comment:9 follow-up: 10 Changed 11 years ago by
comment:10 Changed 11 years ago by
comment:11 Changed 11 years ago by
Summary: | Upgrade Cython to 0.15 → Upgrade Cython to 0.15 or later |
---|
:)
comment:12 Changed 11 years ago by
Description: | modified (diff) |
---|---|
Summary: | Upgrade Cython to 0.15 or later → Upgrade Cython to 0.15.1 |
No further patches needed for 0.15.1.
Changed 11 years ago by
Attachment: | 11761-cython-0.15-rebased.patch added |
---|
comment:13 Changed 11 years ago by
Description: | modified (diff) |
---|---|
Milestone: | sage-4.7.2 → sage-4.7.3 |
Changed 11 years ago by
Attachment: | cython-0.15.1.diff added |
---|
Diff for the new Cython spkg, for review only
comment:14 Changed 11 years ago by
Minor comment: spkg-install
is not executable (I know it is made executable upon building, but better fix this in the spkg anyway).
comment:15 Changed 11 years ago by
Description: | modified (diff) |
---|
comment:16 Changed 11 years ago by
Status: | needs_review → needs_work |
---|
Why is the directory tests
missing from the Cython source? This should either be fixed, or be documented.
comment:17 Changed 11 years ago by
What should be done with:
cdef mpz_t denom mpz_init_set_si(denom, 1)
This code is correct but gives a local variable 'denom' referenced before assignment
warning.
(there are many of these in sage/matrix/misc.pyx
for example)
comment:18 Changed 11 years ago by
Description: | modified (diff) |
---|---|
Reviewers: | → Jeroen Demeyer |
Replying to robertwb:
Yeah!