* cc: sage-combinat, nthiery, bump (added)
* status: new => needs_review
* author: => John Palmieri
Comment:
Here's a patch. The patch fixes all but one of the issues. One question:
in the docstring for {{{apply_simple_projection}}} in
{{{sage.categories.coxeter_groups.py}}}, I changed {{{\opi}}} to
{{{\overline\pi}}} -- this looked like the right change, given the other
documentation. But I would like some confirmation from someone (Nicolas?
Dan?) familiar with that code.
The remaining issue: I don't know what's causing
{{{
<autodoc>:0: (ERROR/3) Unexpected indentation.
<autodoc>:0: (ERROR/3) Unexpected indentation.
<autodoc>:0: (ERROR/3) Unexpected indentation.
}}}
but I'm trying to track it down. A patch could go on another ticket -- no
need to hold up this one.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8492#comment:1>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, and MATLAB
* status: needs_review => positive_review
* reviewer: => Minh Van Nugyen
Comment:
The patch is OK by me. The remaining three warnings can be dealt with in
another ticket.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8492#comment:2>
* status: positive_review => closed
* resolution: => fixed
* merged: => sage-4.3.4.rc0
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8492#comment:3>
Comment(by jhpalmieri):
Replying to [comment:1 jhpalmieri]:
> The remaining issue: I don't know what's causing
{{{
<autodoc>:0: (ERROR/3) Unexpected indentation.
<autodoc>:0: (ERROR/3) Unexpected indentation.
<autodoc>:0: (ERROR/3) Unexpected indentation.
}}}
> but I'm trying to track it down. A patch could go on another ticket --
no need to hold up this one.
See #8511.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8492#comment:4>