#5457: Refactor symmetric functions and k-bounded subspace
----------------------------------------------------------------+-----------
       Reporter:  nthiery                                       |         
Owner:  mhansen                    
           Type:  enhancement                                   |        
Status:  positive_review            
       Priority:  major                                         |     
Milestone:  sage-5.3                   
      Component:  combinatorics                                 |    
Resolution:                             
       Keywords:  symmetric functions, sd38, sd40               |   Work 
issues:                             
Report Upstream:  N/A                                           |     
Reviewers:  Dan Bump, Nicolas M. ThiƩry
        Authors:  Mike Zabrocki, Anne Schilling, Jason Bandlow  |     Merged 
in:                             
   Dependencies:  #11563, #13109, #12969                        |      
Stopgaps:                             
----------------------------------------------------------------+-----------
Changes (by bump):

  * status:  needs_review => positive_review


Comment:

 > I think one of the original reviewers should review this additional
 patch.

 This adds a __cmp__ method for SymmetricFunctions, which was missing, and
 that
 was uncovered by the test in llt.py. Two SymmetricFunctions instances are
 compared equal if and only if they have the same base ring, which is as it
 should be.

 Unless I'm missing something the patch is obviously correct. I ran
 --testall --long
 in the sf directory and all tests passed.

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