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

Mark cuda builds of jaxlib 0.4.30 and 0.4.31 as broken due to jax runtime check on cudnn version #1050

Merged
merged 1 commit into from
Aug 22, 2024

Conversation

traversaro
Copy link
Contributor

jax==0.4.31 has a check at runtime (that is not reflected on cuda metadata) on cudnn>=9, and so trying to use cuda on jax==0.4.31 result in an error like:

CUDA backend failed to initialize: Unable to use CUDA because of the following issues with CUDA components:
Outdated cuDNN installation found.
Version JAX was built against: 8907
Minimum supported: 9100
Installed version: 8907

See conda-forge/jaxlib-feedstock#277 and conda-forge/jax-feedstock#149 .

As the only possible combinations of jaxlib and jax that can be installed together (given their mutual constrained dependencies) is:

  • jax==0.4.31 + jaxlib==0.4.31
  • jax==0.4.31 + jaxlib==0.4.30

I think it make sense to mark as broken the cuda builds of jaxlib==0.4.31 and jaxlib==0.4.30 as they have the constraint of installing cudnn 8.*, and they can't be used with jax==0.4.31.

This will ensure that users that install jaxlib==*=*cuda* to get a cuda-enabled jax actually get a working jax+cuda. This will continue to install a cpu-only version of jax if one just installs conda install jax, to actually fix that we need instead to rebuild jaxlib with cudnn==9, see conda-forge/conda-forge-pinning-feedstock#6310 .

For more details, see the discussion in conda-forge/jax-feedstock#149 (comment) .

As we were discussing on this with @ngam, it would be great to have his feedback before merging.

ping @conda-forge/jaxlib @conda-forge/jax

Checklist:

  • I want to mark a package as broken (or not broken):
    • Added a description of the problem with the package in the PR description.
    • Pinged the team for the package for their input.

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

Successfully merging this pull request may close these issues.

2 participants