#8438: Construction of a skew partition from its row and column lengths
------------------------------+---------------------------------------------
   Reporter:  hivert          |       Owner:  hivert         
       Type:  enhancement     |      Status:  needs_review   
   Priority:  major           |   Milestone:  sage-4.3.4     
  Component:  combinatorics   |    Keywords:  skew partitions
     Author:  Florent Hivert  |    Upstream:  N/A            
   Reviewer:  Nicolas Borie   |      Merged:                 
Work_issues:                  |  
------------------------------+---------------------------------------------
Changes (by hivert):

  * reviewer:  => Nicolas Borie


Comment:

 Replying to [comment:2 nborie]:
 > Can you improve that :
 >
 {{{
 sage: S = SkewPartition(([1,1,1,1,1,1],[1,1,1,1,1,1]))
 sage: S.column_lengths()
 [0]
 sage: S.row_lengths()
 [0, 0, 0, 0, 0, 0]
 sage: from_row_and_column_length([0,0,0,0,0,0],[0])
 ---------------------------------------------------------------------------
 ValueError                                Traceback (most recent call
 last)
 ...
 ...
 ValueError: invalid skew partition
 }}}
 The cases with 0 row and columns length are ambiguous (see the example in
 the doc). I now raise a proper error:
 {{{
 sage: sage: from_row_and_column_length([0,0,0,0,0,0],[0])
 ---------------------------------------------------------------------------
 ValueError                                Traceback (most recent call
 last)
 ...
 ValueError: row and column length must be positive
 }}}
 I re-uploaded the new patch ! Please review.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8438#comment:4>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to