On Sun, 10 Oct 2010 17:53:49 -0700 (PDT)
kcrisman <kcris...@gmail.com> wrote:

> > It looks like the top level binomial() function is a mess already.
> >
> >  - binomial does not accept variable when only in the lower argument
> >        http://trac.sagemath.org/sage_trac/ticket/9634
> >
> >  - binomial does not accept float
> >        http://trac.sagemath.org/sage_trac/ticket/9633
> 
> Oh, yes - several of us have tried to make it work better with various
> input, but likely it could be completely rewritten at this point.
> 
<snip>
> > Replacing the top level binomial function with the symbolic one
> > defined in sage.functions.other should provide an easy fix for all
> > these problems. I wonder if it effects the speed too much.
> 
> I couldn't figure out how to do this at the time, but certainly this
> was a goal in some earlier revisions.  No matter how I imported it,
> the arith.py one is the one that would show up, but this would be a
> great goal.

I uploaded a patch at #9634 which does this:

http://trac.sagemath.org/sage_trac/attachment/ticket/9634/trac_9634-symbolic_binomial.patch

There are still some issues that need work (see [1]), but this should
provide a good start.

[1] http://trac.sagemath.org/sage_trac/ticket/9634#comment:2


Cheers,
Burcin

-- 
To post to this group, send an email to sage-devel@googlegroups.com
To unsubscribe from this group, send an email to 
sage-devel+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/sage-devel
URL: http://www.sagemath.org

Reply via email to