-
Notifications
You must be signed in to change notification settings - Fork 85
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
Drop MXNet Support #593
Comments
how do we know? |
This issue makes it seem like having dedicated distributions with full functionality like TF and PyTorch isn't going to happen right away. The PR that is linked at the very end of it does add sampling (presumably in the next release) but it again lacks the full feature set of the other backends. In my mind it makes more sense to just discontinue any effort on MXNet and think more about PR #377 instead. |
I think it would be a good idea to replace MXNet with jax (#377 ) it's a more fully featured backend that has
and could actually be the one backend that makes things faster than numpy (currently pytorch and tf are only faster in very extreme cases) |
Does jax support distributions? |
@kratsg Given the 👍 I assume you have no qualms with me opening a MR for a branch in which I've already done this? |
yes, no qualms |
Description
MXNet isn't going to be implementing statistical distributions anytime soon, so it seems like they will be diverging from the rest of our ML framework backends even more. Instead of keeping MXNet around as a vestigial dependency I would suggest we just drop support for it.
Thoughts @kratsg @lukasheinrich?
The text was updated successfully, but these errors were encountered: