-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Remove Jax and Haiku #8837
Remove Jax and Haiku #8837
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
… `04-jax-example`
…orch instead of jax
…h-example.ipynb`
19c7115
to
6945153
Compare
…encies. - remove dm-haiku from dependencies
…-exception block in `third_party.py` - move torch back to under data_science marker in setup.cfg
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work @khoaguin as always !! 💯 🎉
|
||
try: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏾
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you very much for the kind words @shubham3121
Description
Since Haiku is deprecated, we should remove Jax and update the notebooks where Jax is used to use Torch
Link to Issue
Affected Dependencies
Notebook Changes
The 5th notebook in
notebooks/api
and the 4 notebooks innotebooks/tutorials/models_training/
are modified to usetorch
instead ofjax
andhaiku