#13248: Coxeter groups: all elements of length N, random elements
-------------------------------------+-------------------------------------
Reporter: sdenton | Owner: joyner
Type: enhancement | Status: needs_review
Priority: minor | Milestone: sage-6.3
Component: group theory | Resolution:
Keywords: sd40, coxeter | Merged in:
Authors: Tom Denton, | Reviewers: Mike Zabrocki, Travis
Frédéric Chapoton | Scrimshaw
Report Upstream: N/A | Work issues:
Branch: u/tscrim/13248 | Commit:
Dependencies: | 707c9c39ec02ba112f19e614a26c99e430b7bd67
| Stopgaps:
-------------------------------------+-------------------------------------
Changes (by tscrim):
* reviewer: Mike Zabrocki => Mike Zabrocki, Travis Scrimshaw
* branch: u/chapoton/13248 => u/tscrim/13248
* commit: fb613e237c7fb1785256f33fe115d7890c6120bb =>
707c9c39ec02ba112f19e614a26c99e430b7bd67
Comment:
Hey Frederic,
I've done some minor tweaks and made it so that `random_element` chooses
the length at random in `[0, 1000]` (so the word is not too long) if not
specified. LGTM otherwise, so if you're happy with my changes, then
positive review.
----
New commits:
||[http://git.sagemath.org/sage.git/commit/?id=ec5652c144d6707d3fd80147d001b4f2157faea8
ec5652c]||{{{Merge branch 'u/chapoton/13248' of trac.sagemath.org:sage
into u/tscrim/13248}}}||
||[http://git.sagemath.org/sage.git/commit/?id=707c9c39ec02ba112f19e614a26c99e430b7bd67
707c9c3]||{{{Some review tweaks.}}}||
--
Ticket URL: <http://trac.sagemath.org/ticket/13248#comment:16>
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.
