#18018: implement componentwise comparison of tableaux
-------------------------------------------------+-------------------------
       Reporter:  opechenik                      |        Owner:
           Type:  defect                         |       Status:  closed
       Priority:  major                          |    Milestone:  sage-6.6
      Component:  combinatorics                  |   Resolution:  fixed
       Keywords:  days64, tableau                |    Merged in:
        Authors:  Oliver Pechenik                |    Reviewers:  Darij
Report Upstream:  N/A                            |  Grinberg
         Branch:                                 |  Work issues:
  ba5143cf8d3207cf08126f74da6e0a8d20bcbe48       |       Commit:
   Dependencies:                                 |     Stopgaps:
-------------------------------------------------+-------------------------
Changes (by vdelecroix):

 * commit:  ba5143cf8d3207cf08126f74da6e0a8d20bcbe48 =>


Comment:

 Hello,

 Did the reviewer really test the code before setting it to positive
 review?!
 {{{
 sage: t = Tableau([[1,2],[3]])
 sage: t.leq(1)
 TypeError: unsupported operand parent(s) for '+':
 'Integer Ring' and '<type 'str'>'
 }}}

 The line
 {{{
              raise TypeError(secondtab + " must be a tableau")
 }}}
 is wrong since only string can be concatenated to strings.

 Vincent

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