#17573: Wrap Gap Structure Description
-------------------------------------+-------------------------------------
Reporter: kcrisman | Owner:
Type: enhancement | Status: needs_review
Priority: minor | Milestone: sage-6.5
Component: group theory | Resolution:
Keywords: | Merged in:
Authors: Sergey Bykov | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/captaintrunky/wrap_gap_structure_description|
3afe347685ec8ccf777c633f9c91bd6f334b1c83
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by ncohen):
Hello again,
I thought for a while about how we could avoid some copy/pate meant to
handle the GAP output, then ended up using a trick. I wrote a couple of
commits which do the following:
1) Move {{{structure_description}}} from `permgroup.py` to `generic.py`
without doing any modification to it
2) Update it a bit (change 'self' into 'G', add a couple of examples) then
bind this function to the {{{permutation_group}}} and
{{{finitely_presented_group}}} classes. And also to {{{Matrix_group}}}
while I was at it.
I had to find a workaround for this `.sage()/.str()` problem you must have
met yourself while you were working on this patch. This code works but
there is definitely something fishy there.
Tell me what you think,
Nathann
P.S.: Those two commits can be found at: `public/17573`
--
Ticket URL: <http://trac.sagemath.org/ticket/17573#comment:12>
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.
For more options, visit https://groups.google.com/d/optout.
