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

[CI] Bump Flax and Jaxlib versions to fix Jaxlib install error #15421

Merged

Conversation

Liam-Sturge
Copy link
Contributor

The Flax dependency Orbax v0.1.8 has enforced the deprecation of installing Orbax as a standalone package. Flax v0.6.8 (which is currently pinned in the script docker/install/ubuntu_install_jax.sh) attempts to install Orbax as a standalone package and raises an error about doing so.

Going forward, the package orbax-checkpoint should be installed instead. Flax v0.6.8 does not recognise this and attempts to install Orbax instead of orbax-checkpoint and the installation fails.

In order to resolve Jax installation issues, bumping the version of Flax to be at least 0.6.9, which resolves the problem. Flax >= 0.6.9 does not pin the version of orbax-checkpoint that it installs and the latest version requires Jax >= 0.4.9 to be installed so the two must be updated together.

Issue: 15420 raised that details the error which occurs and steps to reproduce.

cc @leandron @Mousius @lhutton1 @ekalda @areusch @konturn @tqchen

The Flax dependency Orbax (v0.1.8) has deprecated being able to install
Orbax as a standalone package. Flax v0.6.8 attempts to install Orbax as
a standalone package and raises an error about doing so.

Going forward, the package orbax-checkpoint should be installed instead.
Flax v0.6.8 does not recognize this and attempts to install Orbax
instead of orbax-checkpoint and the installation fails.

In order to resolve Jax installation issues, bumping the version of Flax
to be at least 0.6.9, which resolves the problem. Flax >= 0.6.9 does not
pin the version of orbax-checkpoint that it installs and the latest
version requires Jax >= 0.4.9 to be installed so the two must be updated
together.
@tvm-bot
Copy link
Collaborator

tvm-bot commented Jul 27, 2023

Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from Reviewers by @-ing them in a comment.

Generated by tvm-bot

@Liam-Sturge Liam-Sturge changed the title [CI] Bump Flax and Jax versions to fix Jax install error [CI] Bump Flax and Jaxlib versions to fix Jaxlib install error Jul 27, 2023
Copy link
Member

@yongwww yongwww left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lhutton1 lhutton1 merged commit 64ac43a into apache:main Jul 28, 2023
@lhutton1
Copy link
Contributor

Thanks @Liam-Sturge @yongwww

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants