#13203: Promotion on Tableaux should force its argument to be an Integer.
---------------------------------------+------------------------------------
       Reporter:  nthiery              |         Owner:  sage-combinat    
           Type:  defect               |        Status:  needs_review     
       Priority:  major                |     Milestone:  sage-5.4         
      Component:  combinatorics        |    Resolution:                   
       Keywords:  tableaux, promotion  |   Work issues:                   
Report Upstream:  N/A                  |     Reviewers:  Nicolas M. Thiéry
        Authors:  Frédéric Chapoton    |     Merged in:                   
   Dependencies:                       |      Stopgaps:                   
---------------------------------------+------------------------------------

Comment (by nthiery):

 I just had a look, and this sounds good. For consistency, can you put
 spaces on both sides of the `=` sign, and move this line just after the
 is_rectangular test (putting together all the argument
 checks/preprocessing).

 Once done, you can set a positive review on my behalf (where is the
 patchbot?)

 Ah: it depends syntactically on #13074. I just moved your patch right
 after it in the queue; please run the tests with them applied.


 Thanks!

 Cheers,
                                    Nicolas

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