-
Notifications
You must be signed in to change notification settings - Fork 5
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
Switch from AbstractDifferentiation to DifferentiationInterface? #19
Comments
Yep very open. I would just remove AbstractDifferentiation entirely and bump the major-number version. |
Replace AbstractDifferentiation with DifferentiationInterface #20 This seems great! Is this branch ready for use? The tests failed for me so I thought I would ask. I get a compatibility issue:
|
I just updated that branch to the latest version of DI but I think other compat bounds need to be loosened for tests, most notably that on Turing in |
Great, thanks for that, gonna try to take a look today and hopefully merge! |
Hi there @marius311! My new package DifferentiationInterface is becoming the new standard for autodiff bindings, would you be open to a PR that transitions towards it?
The text was updated successfully, but these errors were encountered: