#11321: Make lcalc compatible with the new PARI
--------------------------------------------+-------------------------------
   Reporter:  jdemeyer                      |          Owner:  jdemeyer         
                             
       Type:  defect                        |         Status:  positive_review  
                             
   Priority:  major                         |      Milestone:  sage-4.7.3       
                             
  Component:  packages                      |       Keywords:  lcalc spkg       
                             
Work_issues:                                |       Upstream:  Reported 
upstream. Developers acknowledge bug.
   Reviewer:  Volker Braun, Leif Leonhardy  |         Author:  Jeroen Demeyer, 
Leif Leonhardy                
     Merged:                                |   Dependencies:  #11130           
                             
--------------------------------------------+-------------------------------
Changes (by was):

  * status:  needs_review => positive_review


Comment:

 I read through the diff to the spkg, and everything looked reasonable to
 me.  I applied the new pari stuff (#11330), then build this spkg, and it
 worked fine, and passed the full test suite (for Sage) after applying it.

 I tried one simple example "by hand" that actually *uses* lcalc, and was
 not pleased by what happened:
 {{{
 sage: E = EllipticCurve('37a')
 sage: L = E.lseries()
 sage: L.zeros(10)
   ***   Warning: new stack size = 1030944 (0.983 Mbytes).
 [0.000000000, 5.00317001, 6.87039122, 8.01433081, 9.93309835, 10.7751382,
 11.7573247, 12.9583864, 15.6038579, 16.1920174]
 sage: L.zeros(10)
   ***   Warning: new stack size = 1030944 (0.983 Mbytes).
 [0.000000000, 5.00317001, 6.87039122, 8.01433081, 9.93309835, 10.7751382,
 11.7573247, 12.9583864, 15.6038579, 16.1920174]
 }}}
 Basically, every time you use lcalc to do anything with elliptic curve
 L-series, you get some mysterious warning (of course, really output from
 PARI).   However, the problem has been in released Sage for a long time,
 so it should be a separate ticket.    It was in sage-4.7.   That's now
 #11985.

 So I say: positive review for this ticket.   Great work guys for cleaning
 up all kinds of little issues.  This is not an easy spkg, to put it
 mildly.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11321#comment:48>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to