google-deepmind / google-deepmind/acme
Unsolvable jax dependency
- Dominant language
- Python
- Stars
- 4.1k
- Forks
- 553
- PR merge metrics
- No merged PRs in 30d
Description
Following the most recent commit, the jax.tree module is now being used instead of jax.tree_*. This is change requires jax >= 0.4.25. However, the are still many parts of the repository that are still old/deprecated. For instance, if you install 0.4.25 you might get something like:
`AttributeError: module 'jax.random' has no attribute 'KeyArray'`
This is because this module has been removed in jax 0.4.24, meaning that in order to not run into this problem you need jax <=0.4.23. Obviously this goes against the requirement above.
Lastly, there is still the issue with DeviceArray and ShardedDeviceArray. They have all been changed to somply jax.Array back in jax=0.4.0! At the current state of the repo you basically need to add lines like:
`jax.interpreters.xla.DeviceArray = jax.Array` in order to be able to even import acme...
Contributor guide
Assessment
This issue has not been assessed yet.