#5537: [with patch, mostly positive review] bug in __cmp__ in
permgroup_element.pyx
--------------------------+-------------------------------------------------
Reporter: jhpalmieri | Owner: robertwb
Type: defect | Status: new
Priority: major | Milestone: sage-3.4.2
Component: group_theory | Keywords:
--------------------------+-------------------------------------------------
Comment(by robertwb):
I'm not sure about the second colon on tests, but I thought that was
needed for the ReST verbatim blocks. Other than that (which, again, I'm
uncertain on) I approve of the referee patch--the explanation of the
ordering is especially good.
The first examples with the generators were there originally. I don't
think they're the best, but I figured we'd keep them at least. As for
whether or not it make sense to order these things, my choice of ordering
was solely to be consistent with what was already there. There is another
discussion ongoing at the moment about whether or not it makes sense to
try and order everything, or just raise errors for non-obviously
comparable things (in the case of permutations, there isn't an obvious
ordering to choose), so I'm not sure its worth investing too much time
into this until the dust settles there.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5537#comment:9>
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
-~----------~----~----~----~------~----~------~--~---
