[sage-devel] Re: AbelianGroup's subgroups

2009-10-04 Thread davidloeffler
Sorry, I've just seen William's earlier post in this thread, and I see that there was no need for my rather combative previous post -- I apologise for any offence I may have caused. David On Oct 4, 2:05 am, William Stein wrote: > On Sat, Oct 3, 2009 at 1:30 PM, David Joyner wrote: > > > The ab

[sage-devel] Re: AbelianGroup's subgroups

2009-10-04 Thread davidloeffler
On Oct 4, 4:54 am, William Stein wrote: > On Sat, Oct 3, 2009 at 8:07 PM, Rob Beezer wrote: > > > On Oct 3, 6:05 pm, William Stein wrote: > >> And this has already been almost completed by David Loeffler based > >> on work by me.  http://trac.sagemath.org/sage_trac/ticket/6449 > > > The work

[sage-devel] Re: AbelianGroup's subgroups

2009-10-03 Thread William Stein
On Sat, Oct 3, 2009 at 8:07 PM, Rob Beezer wrote: > > On Oct 3, 6:05 pm, William Stein wrote: >> And this has already been almost completed by David Loeffler based >> on work by me.  http://trac.sagemath.org/sage_trac/ticket/6449 > > The work at #6449 creates additive abelian groups by extending

[sage-devel] Re: AbelianGroup's subgroups

2009-10-03 Thread Rob Beezer
On Oct 3, 6:05 pm, William Stein wrote: > And this has already been almost completed by David Loeffler based > on work by me.  http://trac.sagemath.org/sage_trac/ticket/6449 The work at #6449 creates additive abelian groups by extending the class for finitely-generated modules over ZZ. Assuming

[sage-devel] Re: AbelianGroup's subgroups

2009-10-03 Thread William Stein
On Sat, Oct 3, 2009 at 1:30 PM, David Joyner wrote: > > The abelian group class needs to be rewritten. The basic idea > was to try to use GAP as much as possible and use > Python/Sage to parse input and output. For various > technical reasons, that did not work as nicely as hoped. > The correct s

[sage-devel] Re: AbelianGroup's subgroups

2009-10-03 Thread David Joyner
The abelian group class needs to be rewritten. The basic idea was to try to use GAP as much as possible and use Python/Sage to parse input and output. For various technical reasons, that did not work as nicely as hoped. The correct solution, may be to rewrite it from scratch completely separate fr