#13762: Implement k-bounded quotient space
---------------------------------------------------+------------------------
       Reporter:  chrisjamesberg                   |         Owner:  
chrisjamesberg  
           Type:  enhancement                      |        Status:  needs_work 
     
       Priority:  major                            |     Milestone:  sage-5.6   
     
      Component:  combinatorics                    |    Resolution:             
     
       Keywords:  symmetric functions, partitions  |   Work issues:  don't use 
assert
Report Upstream:  N/A                              |     Reviewers:  Anne 
Schilling  
        Authors:  Chris Berg, Mike Zabrocki        |     Merged in:             
     
   Dependencies:                                   |      Stopgaps:             
     
---------------------------------------------------+------------------------

Comment (by jdemeyer):

 Replying to [comment:22 zabrocki]:
 > {{{
 > if isinstance(c, Partition_class):
 >       assert len(rest) == 0
 > }}}
 > I don't see how to trigger this.  Is it ok to leave it as is?
 Absolutely. This is how `assert` should be used: to say "this condition is
 always true, without any further assumptions".

 > Some of our assertion errors that appear are triggered in code in
 free_module.py.  Should I change that code here or open a new ticket?
 I don't think you have to change it here, but at least make sure you don't
 introduce any new wrongly-used asserts.

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