Skip to content
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

add ForwardRuleConfigBackend (to match ReverseRuleConfigBackend) #80

Open
gdalle opened this issue Apr 24, 2023 · 7 comments
Open

add ForwardRuleConfigBackend (to match ReverseRuleConfigBackend) #80

gdalle opened this issue Apr 24, 2023 · 7 comments
Labels
feature New feature or request

Comments

@gdalle
Copy link
Member

gdalle commented Apr 24, 2023

If I understand correctly, AbstractDifferentiation is compatible with ChainRules backends that work in reverse mode, but not in forward mode?

@oxinabox
Copy link
Member

The only fully ChainRules compatible forward-mode AD that is working right now is Diffractor.jl which is yet to tag a release. It does have a very nice forwards mode AD though.

ForwardDiff2.jl was abandoned.

ForwardDiff.jl works only via https://github.com/ThummeTo/ForwardDiffChainRules.jl which is opt-in per the rule.

@gdalle
Copy link
Member Author

gdalle commented Apr 24, 2023

Yeah I did have Diffractor in mind, but also a potential future translator from ChainRules to EnzymeRules. I don't think the PR would be hard, I can draft it if there is interest

@ChrisRackauckas
Copy link
Member

ForwardDiff2 was essentially replaced by TaylorDiff.jl which will be using ChainRules for its vector rules.

@oxinabox
Copy link
Member

It would be good to add this

@oxinabox oxinabox changed the title Why ReverseRuleConfigBackend but no ForwardRuleConfigBackend? add ForwardRuleConfigBackend (to match ReverseRuleConfigBackend) Jul 25, 2023
@gdalle
Copy link
Member Author

gdalle commented Jul 26, 2023

It would be good to add this

Could this have something to do with a certain v0.2.0 release that just came out 😇 ?

@gdalle
Copy link
Member Author

gdalle commented Aug 5, 2023

It would be good to add this

I can give it a shot but IIUC we can't test it on 1.9 since we need Diffractor?

@gdalle
Copy link
Member Author

gdalle commented Aug 5, 2023

Also, the Diffractor backend already exists without being a special case of the (yet-to-be-written) ForwardRuleConfigBackend, is that an issue?

@gdalle gdalle added the feature New feature or request label Aug 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants