#5664: [with patch, needs work] Bugs in PermutationGroup_subgroup.__cmp__
--------------------------+-------------------------------------------------
Reporter: SimonKing | Owner: SimonKing
Type: defect | Status: new
Priority: critical | Milestone: sage-3.4.1
Component: group_theory | Keywords: comparison subgroup
--------------------------+-------------------------------------------------
Comment(by wdj):
Almost a positive review.
Seems to apply fine to 3.4 and test okay.
The problem is with the docstrings. The docstring for __cmp__
in the subgroup class defines H but never uses it. Plase
(a) improve the docstring for __cmp__
in the subgroup class to include instances of the behavious discussed
above
(b) please add your name to the AUTHOR list at the top
A positive review after that.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5664#comment:11>
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
-~----------~----~----~----~------~----~------~--~---
