#7371: rename quotient_group() to quotient() in groups/perm_gps/permgroup.py
----------------------------+-----------------------------------------------
Reporter: AlexGhitza | Owner: joyner
Type: defect | Status: new
Priority: minor | Milestone: sage-4.2.1
Component: group_theory | Keywords:
Work_issues: | Author:
Reviewer: | Merged:
----------------------------+-----------------------------------------------
There is a generic group method called {{{quotient}}}, which is meant to
return the quotient group by a normal subgroup, and is meant to be
overridden by inheriting classes. However, the corresponding method for
permutation groups is called {{{quotient_group}}} instead:
{{{
sage: S = SymmetricGroup(6)
sage: N = S.normal_subgroups()[1]
sage: S.quotient(N)
---------------------------------------------------------------------------
NotImplementedError Traceback (most recent call
last)
/home/ghitza/.sage/temp/artin/674/_home_ghitza__sage_init_sage_0.py in
<module>()
/home/ghitza/sage-stable/local/lib/python2.6/site-
packages/sage/groups/group.so in sage.groups.group.Group.quotient
(sage/groups/group.c:1719)()
NotImplementedError:
sage: S.quotient_group(N)
Permutation Group with generators [(), (1,2)]
}}}
The attached patch renames the permutation group method to {{{quotient}}}
and deprecates {{{quotient_group}}}.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7371>
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
-~----------~----~----~----~------~----~------~--~---
