#17995: ascii_art in Tableaux.global_options has outdated documentation
-------------------------------------+-------------------------------------
       Reporter:  MariaMonks         |        Owner:
           Type:  defect             |       Status:  needs_work
       Priority:  major              |    Milestone:  sage-6.6
      Component:  combinatorics      |   Resolution:
       Keywords:  days64, tableau    |    Merged in:
        Authors:  Maria Monks        |    Reviewers:
  Gillespie, Oliver Pechenik         |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:  public/17995       |  ddb4c16f3353186c57316bae94f887027f1557ec
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by tscrim):

 * status:  positive_review => needs_work


Comment:

 First, Darij, reviewer name.

 Second, you should set `.boxes` as an alias for `table` and `plain` for
 `diagram` by adding `alias=dict(plain='diagram', boxes="table")`, see the
 options above the `ascii_art` for instance. This way you need far less
 code for the same effect you're trying to achieve (and it's easier to
 maintain).

 Unfortunately, due to the current limitations of the doctesting framework,
 it doesn't know to run those tests in (dynamically) generated doc. See
 #14272. Also you'll have to update `doctest/sources.py` and it has a test
 to catch exactly how many tests we are currently skipping.

--
Ticket URL: <http://trac.sagemath.org/ticket/17995#comment:7>
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.

Reply via email to