#9621: Fix GAP interface problem in sylow_subgroup method
----------------------------+-----------------------------------------------
Reporter: SimonKing | Owner: joyner
Type: defect | Status: new
Priority: major | Milestone: sage-4.5.2
Component: group_theory | Keywords: GAP string representation
Author: Simon King | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
----------------------------+-----------------------------------------------
The following was reported by [http://groups.google.com/group/sage-
support/browse_thread/thread/9fbca4e4dbadbe37 Kenny Brown]:
{{{
sage: n = 3^2 * 7^2
sage: G = CyclicPermutationGroup(n)
sage: G.sylow_subgroup(3)
Traceback (most recent call last):
...
}}}
The problem is that in the sylow_subgroup method, it is attempted to get
the string presentation of a permutation in GAP by calling gap.eval(...).
However, GAP truncates the output. So, better use gap.eval('Print(...)')
instead.
Moreover, the method uses quite generic variable names in the GAP
interface. This is dangerous, as the use of variable names that any
average user might choose as well can have nasty side effects.
The attached patch fixes both problems.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9621>
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.
