-
Notifications
You must be signed in to change notification settings - Fork 37
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
[wip] New Diff2 #521
base: master
Are you sure you want to change the base?
[wip] New Diff2 #521
Conversation
mm, reading this will break compatibility with SMT, so this don't is an option D: |
Thinking, the diff of SMT diverge of the diff of SymPy, maybe can be good idea move this diff to diff2 to better compatibility with SymPy. In what diverge? diff of sympy use chain rule, while diff of SMT only partial derive of the first founded var. |
ups, don't is a octave problem, my bad again, i notice this now, the actual diff don't its compatible with octave D: |
Regarding your specific example: that does look wrong, I think symvar should choose |
Now: I don't understand what you're trying to fix. Was there something else you're trying to address here? |
Hi!, it feels to much time, well here i have a rewrite to the diff function, the reasons are simple, first, when we have a function with more than one var the diff command send the partial derivation only of the first founded var, ex:
i think this is really bad, theoretically the diff should use the chain rule and apply it, but actually we can't do that, at least while this issue is open: #519
So, i think its better have a warning/error instead a wrong calc, (and start searching the problem etc).
I try keep the main functions of diff, so the new syntax would be :
this code recognize symbols on functions if you don't send the first derive var, and if the first arg is a number is derived in that grade, one of the differences with the actual diff is this:
actual diff:
new diff
basically the new diff(f, 2, x) menas, derive f two times in x, then derive it in x again.
I think include the chain rule but i will leave that to sympy: sympy/sympy#11573
Thx. Cya.