#11360: The Rubik's Cube group in Sage doesn't support many basic operations
for a
dumb reason.
------------------------------------+---------------------------------------
Reporter: was | Owner: jason, ncohen, rlm
Type: enhancement | Status: needs_review
Priority: minor | Milestone: sage-5.10
Component: graph theory | Resolution:
Keywords: | Work issues:
Report Upstream: N/A | Reviewers: Nathann Cohen
Authors: Travis Scrimshaw | Merged in:
Dependencies: | Stopgaps:
------------------------------------+---------------------------------------
Changes (by {'newvalue': u'Travis Scrimshaw', 'oldvalue': ''}):
* status: needs_work => needs_review
* author: => Travis Scrimshaw
Old description:
> We have
> {{{
> sage: G = CubeGroup()
> sage: G.order()
> Traceback (most recent call last):
> ...
> AttributeError: 'CubeGroup' object has no attribute '_gap_string'
> sage: G.sylow_subgroup(3)
> Traceback (most recent call last):
> ...
> AttributeError: 'CubeGroup' object has no attribute '_gap_string'
> }}}
>
> However, these things are all easily computable in Sage:
> {{{
> sage: G2 = PermutationGroup(G.gens())
> sage: G2.order()
> 43252003274489856000
> sage: G2.sylow_subgroup(3)
> Permutation Group with generators [(24,30,43)(32,48,38), ...
> }}}
>
> Apply:
> * [attachment:trac_11360-cubegroup.patch]
> * [attachment:trac_11360-cubegroup2.patch]
New description:
We have
{{{
sage: G = CubeGroup()
sage: G.order()
Traceback (most recent call last):
...
AttributeError: 'CubeGroup' object has no attribute '_gap_string'
sage: G.sylow_subgroup(3)
Traceback (most recent call last):
...
AttributeError: 'CubeGroup' object has no attribute '_gap_string'
}}}
However, these things are all easily computable in Sage:
{{{
sage: G2 = PermutationGroup(G.gens())
sage: G2.order()
43252003274489856000
sage: G2.sylow_subgroup(3)
Permutation Group with generators [(24,30,43)(32,48,38), ...
}}}
Apply:
* [attachment:trac_11360-cubegroup-ts.patch]
--
Comment:
Here's a new version of the patch which fixes the problem by making
`CubeGroup` properly use OOP by calling its superclass's `__init__` (and
subsequently got rid of the `_group` attribute). While I was at it, I
brought the file up to full coverage.
For patchbot:
Apply: trac_11360-cubegroup-ts.patch
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11360#comment:10>
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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.
