#14014: Update matrix groups to new Parents, libGAP.
--------------------------------------------------+-------------------------
Reporter: vbraun | Owner: joyner
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-5.10
Component: group theory | Resolution:
Keywords: | Work issues:
Report Upstream: N/A | Reviewers: David Roe
Authors: Volker Braun | Merged in:
Dependencies: #14187, #14323, #14284, #14640 | Stopgaps:
--------------------------------------------------+-------------------------
Old description:
> This patch adds support for cyclotomics and matrices to LibGAP. And
> rewrites the matrix groups on top of that.
>
> * Apply [attachment:trac_14014_libgap_cyclotomic_matrix.patch]
> * Apply [attachment:trac_14014_deletions.patch]
> * Apply [attachment:trac_14014_parents_for_matrix_groups.patch]
> * Apply [attachment:trac_14014_parents_group_dependents.patch]
> * Apply [attachment:trac_14014_iterator.patch]
> * Apply [attachment:trac_14014_misc.patch]
>
> Follow-up: #14039.
New description:
This patch adds support for cyclotomics and matrices to LibGAP. And
rewrites the matrix groups on top of that.
* Apply [attachment:trac_14014_libgap_cyclotomic_matrix.patch]
* Apply [attachment:trac_14014_deletions.patch]
* Apply [attachment:trac_14014_parents_for_matrix_groups.patch]
* Apply [attachment:trac_14014_parents_group_dependents.patch]
* Apply [attachment:trac_14014_iterator.patch]
* Apply [attachment:trac_14014_misc.patch]
* Apply [attachment:trac_14014-review-ts.patch]
Follow-up: #14039.
--
Comment (by tscrim):
Hey Volker,
I was talking about the modules `libs.gap.*`, in particular the one which
triggers the rest is `libs.gap.element`. However by lazily importing this,
I ended up getting errors, so I don't think we can get around this (plus
it's not really a big import or a significant slowdown to startup time).
I'd guess this is related to #14357, but it's definitely not worth holding
this ticket up.
Anyways, I've uploaded a small reviewer patch which does some minor to
trivial docstring changes to methods that are being moved around by this
patch. If you're happy with my tweaks, I think we can set this to positive
review.
Best,[[BR]]
Travis
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14014#comment:42>
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.
