#8899: Implement non commutative symmetric functions
--------------------------------------------------------------------------------+
       Reporter:  nthiery                                                       
|         Owner:  sage-combinat                             
           Type:  enhancement                                                   
|        Status:  closed                                    
       Priority:  major                                                         
|     Milestone:  sage-5.4                                  
      Component:  combinatorics                                                 
|    Resolution:  fixed                                     
       Keywords:  sd40                                                          
|   Work issues:                                            
Report Upstream:  N/A                                                           
|     Reviewers:  Mike Zabrocki, Franco Saliola, Mike Hansen
        Authors:  Jason Bandlow, Chris Berg, Franco Saliola, Nicolas M. ThiƩry  
|     Merged in:  sage-5.4.beta1                            
   Dependencies:  #12953, #12956, #12959, #13238, #13243, #5457                 
|      Stopgaps:                                            
--------------------------------------------------------------------------------+

Comment (by nthiery):

 Replying to [comment:51 jdemeyer]:
 > This patch abuses `assert` and `AssertionError`. `assert` should not be
 used for control flow. An `assert` checks something which should always be
 true, a failed assertion is always a bug in the program.
 >
 > For example:
 > {{{
 >                 sage: R = NonCommutativeSymmetricFunctions(QQ).R()
 >                 sage: R.skew([2,1], [1])
 >                 Traceback (most recent call last):
 >                 ...
 >                 AssertionError: x must be an element of Non-Commutative
 Symmetric Functions over the Rational Field
 > }}}
 > This is a simple user mistake, for which `assert` is not right.
 >
 > I think this must be fixed.

 I guess that's ok after all. See the same comment on  #5457.

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