#20768: Allow pp of tableau to work with entries with more than 3 digits
-------------------------------------+-------------------------------------
       Reporter:  tscrim             |        Owner:  sage-combinat
           Type:  defect             |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-7.3
      Component:  combinatorics      |   Resolution:
       Keywords:  days74             |    Merged in:
        Authors:  Travis Scrimshaw   |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  public/combinat/fix_tableau_pp-20768|  
70122483465001781c73c8805151e208e880d238
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by chapoton):

 * maybe in
 {{{
 str_tab = reversed(str_tab)
 }}}
 use instead `.reverse` ?

 * maybe in
 {{{
 +        str_tt = [T._repr_diagram().split('\n') for T in self]
 +        if TableauTuples.global_options('convention') == "French":
 +            for T_str in str_tt:
 +                T_str.reverse()
 +        widths = [len(T_str[0]) for T_str in str_tt]
 }}}
 compute the widths before doing the French reversion ? otherwise T_str[0]
 may be short ?

--
Ticket URL: <http://trac.sagemath.org/ticket/20768#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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to