#5159: [with patches, needs review] Add functionality to Galois groups
---------------------------+------------------------------------------------
Reporter: davidloeffler | Owner: davidloeffler
Type: enhancement | Status: assigned
Priority: major | Milestone: sage-3.4.2
Component: number theory | Keywords: galois groups, number theory
---------------------------+------------------------------------------------
Comment(by davidloeffler):
Sorting the list exposed another bug, which I refuse to accept any
responsibility for whatsoever: there is something funny going on in the
default __cmp__ routine in sage.structure.element.Element, which meant
that if x,y were GaloisGroupElements then cmp(x, y) worked but sorted([x,
y]) didn't.
I've worked around this by implementing __cmp__ directly for
GaloisGroupElements. This gets the doctests in doc/en/bordeaux_2008 to
pass on 32bit and 64bit.
Apologies that these things didn't get spotted earlier, but I'm working
from home on a clapped out old 32-bit laptop, on which running sage
-testall takes over three hours, so when I was working on this I only ran
the doctests in sage/rings/number_field. That's a mistake I won't make
again.
David
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5159#comment:23>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
