-
-
Notifications
You must be signed in to change notification settings - Fork 567
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
conversion of binomial fails #6197
Comments
comment:1
The error for the original bug is now
so the fix for this is to create a symbolic binomial and provide for this in the init method, the way factorial is done in functions/other.py (this works for Maxima; I don't have access to the others - but the fix would presumably be identical). However, I can't get this to import properly so that the symbolic binomial is what you get from binomial? like you do for factorial?, even after adding it to the import list in functions/all.py, so I am reluctant to post anything at this point. |
comment:2
This should fix the problem reported in the bug, also hopefully for Mathematica (and for factorial). It doesn't really help that much since Maxima doesn't automatically simplify binomials :) but that wasn't the ticket. Adding Maple to the list, or other functions one desires, would be very similar. The symbolic binomial wouldn't import properly, so this patch does not import it at all, but the bug is still fixed and the tests for these and a few related files pass. |
comment:3
Looks good to me. |
Author: Karl-Dieter Crisman |
Reviewer: Mike Hansen |
comment:4
I received a doctest failure with the patch
|
comment:5
Adding a new function changes the list random_expr uses. This is actually fixed in #6636, though I guess it must have already been a problem in this one. I will try to separate that out from that patch and post updates to both. |
comment:6
Attachment: trac_6197-binomial-maxima.patch.gz Okay, this should do it. Literally the only change is that doctest, so hopefully just testing that should restore positive review. |
comment:7
Looks good. |
Merged: Sage 4.1.2.alpha2 |
Converting symbolic expressions containing binomial coefficients
to maxima elements does not work.
Also the conversions to mathematica and maple fail:
Component: symbolics
Author: Karl-Dieter Crisman
Reviewer: Mike Hansen
Merged: Sage 4.1.2.alpha2
Issue created by migration from https://trac.sagemath.org/ticket/6197
The text was updated successfully, but these errors were encountered: