#13072: Implementation of PartitionTuple + some minor fixes to partition.py
----------------------------------------+-----------------------------------
       Reporter:  andrew.mathas         |         Owner:  Andrew Mathas   
           Type:  enhancement           |        Status:  needs_work      
       Priority:  major                 |     Milestone:  sage-5.4        
      Component:  combinatorics         |    Resolution:                  
       Keywords:  tuples of partitions  |   Work issues:                  
Report Upstream:  N/A                   |     Reviewers:  Travis Scrimshaw
        Authors:  Andrew Mathas         |     Merged in:                  
   Dependencies:  #9265, #11446         |      Stopgaps:                  
----------------------------------------+-----------------------------------

Comment (by andrew.mathas):

 Hi Travis,

 I have uploaded a new version of the patch which fixes the errant comma
 and the indentation in random_element but I think that the
 {{{``(k,r,c)``}}} in the hook_length function is correct because k,r and c
 are arguments to this function. Of course, it would be more correct to
 write {{{(``k``,``r``,``c``)}}} but ReST complains about this. I guess
 that {{{(``k``, ``r``, ``c``)}}} might be OK, although I suspect that it
 will want a few more spaces like {{{( ``k`` , ``r`` , ``c`` )}}} to be
 legal syntax. I prefer using {{{``(k,r,c)``}}} as these three integers
 together comprise a cell which is really a "single" variable... Let me
 know if you disagree.

 Cheers,
 Andrew

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