Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade PARI to 2.7.1 #15767

Closed
jdemeyer opened this issue Jan 30, 2014 · 196 comments
Closed

Upgrade PARI to 2.7.1 #15767

jdemeyer opened this issue Jan 30, 2014 · 196 comments

Comments

@jdemeyer
Copy link

Upstream: http://pari.math.u-bordeaux.fr/pub/pari/unix/pari-2.7.1.tar.gz

Updated (changes have always been made directly in the source tarball and the package will be removed anyway in #15808): http://boxen.math.washington.edu/home/jdemeyer/spkg/genus2reduction-20140211.tar.bz2

Depends on #15888
Depends on #15855

CC: @jpflori @pjbruin @SnarkBoojum

Component: packages: standard

Author: Jeroen Demeyer

Branch/Commit: bf435f8

Reviewer: Peter Bruin

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

@pjbruin
Copy link
Contributor

pjbruin commented Jan 30, 2014

comment:1

Good plan; there have been quite a few changes (also in the interface) from 2.5 to 2.6, but 2.7 (being a stable release) will be close to 2.6.2.

@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@jdemeyer
Copy link
Author

jdemeyer commented Feb 1, 2014

Changed dependencies from #15765 to #15765, #15774

@jdemeyer
Copy link
Author

jdemeyer commented Feb 9, 2014

Branch: u/jdemeyer/ticket/15767

@jdemeyer
Copy link
Author

jdemeyer commented Feb 9, 2014

Author: Jeroen Demeyer

@jdemeyer
Copy link
Author

jdemeyer commented Feb 9, 2014

New commits:

17f864cRemove mp.c.patch; instead call init_memory_functions()
ad63dafMove PARI databases out of PARI package
1678b89Merge remote-tracking branch 'origin/develop' into ticket/15765
bfdad13Merge branch 'u/jdemeyer/ticket/15765' of git://trac.sagemath.org/sage into ticket/15774
0595a42Upgrade to PARI 2.6.2

@jdemeyer
Copy link
Author

jdemeyer commented Feb 9, 2014

Commit: 0595a42

@jdemeyer

This comment has been minimized.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 9, 2014

Changed commit from 0595a42 to 6de648b

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 9, 2014

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

6de648bUpgrade to PARI 2.6.2

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 9, 2014

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

5b823f1Upgrade to PARI 2.6.2

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 9, 2014

Changed commit from 6de648b to 5b823f1

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 9, 2014

Changed commit from 5b823f1 to 5cb3478

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 9, 2014

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

5cb3478Upgrade to PARI 2.6.2

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 9, 2014

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

84c081cUpgrade to PARI 2.6.2

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 9, 2014

Changed commit from 5cb3478 to 84c081c

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 9, 2014

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

8e2cc9eUpgrade to PARI 2.6.2

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 9, 2014

Changed commit from 84c081c to 8e2cc9e

@jdemeyer
Copy link
Author

jdemeyer commented Feb 9, 2014

comment:11

This version compiles fine, but there are about a million doctest failures :-(

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 9, 2014

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

19b2936Upgrade to PARI 2.6.2

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 9, 2014

Changed commit from 8e2cc9e to 19b2936

@jdemeyer
Copy link
Author

jdemeyer commented Feb 9, 2014

comment:13

This is a serious regression:

sage: EllipticCurve('90c3').integral_points(both_signs=False)
[(-15 : 7 : 1), (1 : 39 : 1), (21 : 79 : 1)]

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 9, 2014

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

52e24d0Upgrade to PARI 2.6.2

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 9, 2014

Changed commit from 19b2936 to 52e24d0

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 9, 2014

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

b780916Upgrade to PARI 2.6.2

@pjbruin
Copy link
Contributor

pjbruin commented Aug 19, 2014

comment:127

Replying to @jdemeyer:

Peter, could you please add some documentation for the application of Sturm's Theorem in ell.gp? I checked the calculations by hand and it looks correct, but it would be nice to have the calculation as comment in the file.

Done in the above commit.

@jdemeyer
Copy link
Author

comment:128

Replying to @pjbruin:

Done in the above commit.

Nice.

@vbraun
Copy link
Member

vbraun commented Aug 19, 2014

comment:129

I'm getting merge failures, will release beta1 soon. Please merge with that release then.

@jdemeyer
Copy link
Author

comment:130

Replying to @vbraun:

I'm getting merge failures, will release beta1 soon. Please merge with that release then.

OK, I will do that. Unfortunately, this ticket tends to give a lot of conflicts because it touches so much code (in a way which cannot really be avoided). For beta2, I hope you will give this ticket priority and ask other branches to merge with this.

@vbraun
Copy link
Member

vbraun commented Aug 20, 2014

comment:131

I've released beta1, if you can resolve the conflicts soon then I'll merge this as the first ticket.

@jdemeyer
Copy link
Author

Changed branch from u/pbruin/15767-pari-2.7.1 to u/jdemeyer/ticket/15767

@jdemeyer
Copy link
Author

comment:133

Done. I'm running doctests now to check that nothing got broken.

@jdemeyer
Copy link
Author

Changed commit from 62ca821 to bf435f8

@pjbruin
Copy link
Contributor

pjbruin commented Aug 20, 2014

comment:134

This branch agrees (up to whitespace) with a merge I did. If it passes doctests, you can set it back to positive review.

@vbraun
Copy link
Member

vbraun commented Aug 20, 2014

Changed branch from u/jdemeyer/ticket/15767 to bf435f8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants