#17573: Wrap Gap Structure Description
-------------------------+-------------------------------------------------
Reporter: | Owner:
kcrisman | Status: positive_review
Type: | Milestone: sage-6.5
enhancement | Resolution:
Priority: minor | Merged in:
Component: group | Reviewers: Nathann Cohen
theory | Work issues:
Keywords: | Commit:
Authors: | 294041b72adf4b10f44f913d7439c33c96524eee
Sergey Bykov | Stopgaps:
Report Upstream: N/A |
Branch: |
public/17573 |
Dependencies: |
-------------------------+-------------------------------------------------
Comment (by vbraun):
Replying to [comment:28 ncohen]:
> I took it as a hint that there will never be a working `.gap()` function
on all groups.
Yes. Another possible generic implementation is to call
`.as_permutation_group().structure_description()`, if that is available.
But in some cases you'll have to raise `NotImplementedError`. It is then
up to author of that particular group to override `structure_description`.
> Thus this patch should only move that function from one place to
another. In which class do you think it should go?
Evidently it should be in sage.groups.group.Group
--
Ticket URL: <http://trac.sagemath.org/ticket/17573#comment:29>
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.
