#7096: bug in dual isogeny computation
-------------------------------+--------------------------------------------
   Reporter:  cremona          |       Owner:                        
       Type:  defect           |      Status:  positive_review       
   Priority:  major            |   Milestone:  sage-4.2.1            
  Component:  elliptic curves  |    Keywords:  elliptic curve isogeny
Work_issues:                   |      Author:                        
   Reviewer:                   |      Merged:                        
-------------------------------+--------------------------------------------
Changes (by cremona):

  * status:  needs_review => positive_review


Comment:

 OK, I relented and have reviewed the patch.  It is great.  It applies fine
 to 4.2 and builds and tests ok.  (Even better, my not-yet-complete patch
 for #6887 applied OK on top of this, so a large piece of work which uses
 isogenies a lot still works fine;  that is a very good test, I think).  I
 also built and read through the documentation, which is good -- better
 than before in several respects.  I may have spotted a typo or too but did
 not note them down so will not delay things by hunting them down.

 Positive review: and gives me a spur to finish #6887.

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