#10335: Add domains for permutation groups
-----------------------------+----------------------------------------------
Reporter: mhansen | Owner: joyner
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-5.0
Component: group theory | Keywords:
Work_issues: | Upstream: N/A
Reviewer: Robert Miller | Author: Mike Hansen
Merged: | Dependencies:
-----------------------------+----------------------------------------------
Changes (by rbeezer):
* status: needs_review => needs_work
Comment:
I ran {{{make ptestlong}}} and got quite a few failures. I even tried it
twice - not sure what is up.
* Many were in sage/doc
* Many were simply the new version of {{{_repr_}}} for the symmetric
group, and its subgroups
* The {{{latex()}}} method is putting a comma between the cycles within
each generator. Is this desired?
{{{
return ",".join(["(" + ",".join([latex(x) for x in cycle])+")" for cycle
in self.cycle_tuples()])
}}}
At a minimum it is causing doctest failures.
* 73 failures in sage/rings/number_field/galois_group.py, likely all the
same root cause
Saw these in the patch itself:
* a stray "deprecated_function_alias" in a docstring
* {{{``selfs``}}} in a docstring
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10335#comment:14>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.