#14496: unify the three implementations of gaussian q-binomial coefficients
-------------------------------------+--------------------------------------
       Reporter:  chapoton           |         Owner:  tbd                      
       
           Type:  enhancement        |        Status:  needs_work               
       
       Priority:  major              |     Milestone:  sage-5.10                
       
      Component:  combinatorics      |    Resolution:                           
       
       Keywords:  gaussian binomial  |   Work issues:                           
       
Report Upstream:  N/A                |     Reviewers:  Francis Clarke, Travis 
Scrimshaw
        Authors:  Frédéric Chapoton  |     Merged in:                           
       
   Dependencies:                     |      Stopgaps:                           
       
-------------------------------------+--------------------------------------
Changes (by fwclarke):

  * status:  needs_review => needs_work


Comment:

 I was about to say positive review, when I noticed two things:

 1.  `gaussian_polynomial(5, 2, 7r)` fails.  But this caused by a feature
 of `cyclotomic_value`, so should probably be ignored.

 2.  `gaussian_binomial(4, 2, Zmod(6)(2), algorithm='naive')` fails.  This
 can be very simply rectified by excepting a `ZeroDivisionError` as well as
 a `TypeError` just before the "last attempt".

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14496#comment:21>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to