#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 ncohen):
Yo!
> Really the SX question boils down to : why is `structure_description`
not consistently available for every group, after all it makes sense for
every group.
True. There are oddities in the code, however, like this `MatrixGroup_gap`
class whose description reads:
{{{Matrix group over a ring that GAP understands}}}
I took it as a hint that there will never be a working `.gap()` function
on all groups.
> Just monkey-patching it to yet another group addresses one particular
symptom, but does nothing against the disease.
Indeed. It only makes the method available on some classes where it was
not.
> I agree with Karl-Dieter, just raise `NotImplementedError` where
appropriate (e.g. there is no `gap` method).
Thus this patch should only move that function from one place to another.
In which class do you think it should go?
Nathann
--
Ticket URL: <http://trac.sagemath.org/ticket/17573#comment:28>
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.
