#18743: Display does not support the 'ascii_art' option of _repr_option
-------------------------------------+-------------------------------------
       Reporter:  tscrim             |        Owner:  tscrim
           Type:  defect             |       Status:  needs_work
       Priority:  major              |    Milestone:  sage-6.8
      Component:  interfaces         |   Resolution:
       Keywords:  fancy repr         |    Merged in:
        Authors:  Travis Scrimshaw   |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  public/interfaces/repr_option_ascii_art-18743|  
20bd15dd22b857e41bbe50244112f6128c8082bb
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by tscrim):

 * cc: elixyre, jpflori, SimonKing (added)
 * status:  needs_review => needs_work


Comment:

 I am completely dumbfounded as to why this addition causes the failing
 doctests in `interfaces/psage.py`. It fails in the ''original'' code
 {{{
 ascii_art_repr = ascii_art_repr or
 o.parent()._repr_option('element_ascii_art')
 }}}
 but only when I've added the line before it! I know I could simply add
 `NonImplementedError` or a general `Exception` as the `except` types, but
 this makes me feel like I'm covering up errors that should be allowed to
 propagate.

 Jean-Pierre and Simon, I've cc-ed you incase you had any insight as you
 were the most recent listed authors to `interfaces/expect.py` (and have
 made significant changes and are not currently cc-ed).

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