#10335: Add domains for permutation groups
-----------------------------+----------------------------------------------
Reporter: mhansen | Owner: joyner
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-4.7.2
Component: group theory | Resolution:
Keywords: sd31 | Work_issues: Solaris doctest error
Upstream: N/A | Reviewer: Robert Miller, Rob Beezer,
Nicolas Borie, Nicolas M. Thiéry
Author: Mike Hansen | Merged:
Dependencies: #10334 |
-----------------------------+----------------------------------------------
Changes (by davidloeffler):
* status: new => needs_review
Old description:
> Often, one wants to define a permutation group on a set other than {{{
> {1,2,...,n} }}} such as say {{{ {'a', 'b', 'c', 'd'} }}}.
>
> For example,
>
> {{{
> sage: G = PermutationGroup([ [('c','d')], [('a','c')] ])
> sage: G.orbit('a')
> ['a', 'c', 'd']
> }}}
>
> '''Apply:'''
> 1. [attachment:trac_10335-permgroup_domain-mh.patch]
> 1. [attachment:trac_10335-permgroup_domain-import_loop_fix-nt.patch]
> 1. [attachment:trac_10335-tutorial-languages-fixes.patch]
New description:
Often, one wants to define a permutation group on a set other than {{{
{1,2,...,n} }}} such as say {{{ {'a', 'b', 'c', 'd'} }}}.
For example,
{{{
sage: G = PermutationGroup([ [('c','d')], [('a','c')] ])
sage: G.orbit('a')
['a', 'c', 'd']
}}}
'''Apply:'''
1. [attachment:trac_10335-permgroup_domain-mh.patch]
1. [attachment:trac_10335-permgroup_domain-import_loop_fix-nt.patch]
1. [attachment:trac_10335-tutorial-languages-fixes.patch]
1. [attachment:trac_10335-solaris_issue.patch]
--
Comment:
I don't have access to a Solaris machine to test this on; but, looking at
the code, it's clear that (a) this Sage function just wraps the
corresponding one in Gap, (b) the answer returned on Solaris is just as
much a valid answer as that returned on other systems, and (c) there is
already a perfectly good doctest for that function.
I strongly suggest that we change this doctest to something less
sensitive, given that there is a huge pile of other code waiting on this.
I have just uploaded a patch which does this.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10335#comment:42>
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.
