-
Notifications
You must be signed in to change notification settings - Fork 21
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 backend dispatchers for operations in funsor.ops #61
Comments
This seems OK as long as |
This only gives us flexibility to do things like clamping or have some custom defined behavior, when it might be needed. When the API is exactly the same e.g. in the case of |
This seems like a good idea, also using Note that #55 and #63 implement an |
Another thing I like about this approach is that it would allow us to avoid importing |
Closed by #74 |
It seems to me that the simplest way to support multiple backends in
funsor.ops
would be to use multiple dispatch as follows:What do you think? Will that cause any inadvertent complications otherwise?
The text was updated successfully, but these errors were encountered: