#6449: Additive abelian groups
--------------------------------------------+-------------------------------
   Reporter:  davidloeffler                 |       Owner:  joyner       
       Type:  defect                        |      Status:  needs_work   
   Priority:  major                         |   Milestone:  sage-4.5     
  Component:  group_theory                  |    Keywords:  abelian group
     Author:  David Loeffler                |    Upstream:  N/A          
   Reviewer:  John Cremona, Jim Stankewicz  |      Merged:               
Work_issues:                                |  
--------------------------------------------+-------------------------------

Comment(by davidloeffler):

 Replying to [comment:13 cremona]:
 > Sorry about that.  There was only trivial rebasing needed to the first
 patch (it fixed some small bugs in mrange, and those had already been
 fixed elsewhere, so it was possible to just ignore the failing hunks).

 Yes, that part of the patch got factored out as #6561.

 > There is one doctest failure [...]
 > but I don't actually know what is going on there as it's an indirect
 doctest!

 Hmm. I see what's happening: that doctest assumes that elliptic curve
 torsion subgroups are an instance of the old {{{AbelianGroup}}} class,
 while the new patch makes them an instance of my new
 {{{AdditiveAbelianGroup}}}. Patch coming up.

 David

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