#6616: [with patch; needs review] refactor heegner points code out of
ell_rational_field and support computing higher heegner points
-----------------------------+----------------------------------------------
   Reporter:  robertwb       |       Owner:  was            
       Type:  enhancement    |      Status:  positive_review
   Priority:  major          |   Milestone:  sage-4.3.1     
  Component:  number theory  |    Keywords:                 
Work_issues:                 |      Author:                 
   Upstream:  N/A            |    Reviewer:  John Cremona   
     Merged:                 |  
-----------------------------+----------------------------------------------
Changes (by cremona):

  * status:  needs_review => positive_review


Comment:

 Positive review after testing on 32-bit, and also fixing lots of
 documentation glitches (in the reviewer's patch).  These were mainly found
 by actually running "sage -docbuild all html" and reading the error
 messages (hint hint).  Most common error was double :: where they
 shouldn't be.

 Great work!

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6616#comment:20>
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