Skip to content

jaxlib v0.7.1 - #321

Merged
h-vetinari merged 28 commits into
conda-forge:mainfrom
regro-cf-autotick-bot:0.7.1_h1f8390
Sep 21, 2025
Merged

jaxlib v0.7.1#321
h-vetinari merged 28 commits into
conda-forge:mainfrom
regro-cf-autotick-bot:0.7.1_h1f8390

Conversation

@regro-cf-autotick-bot

Copy link
Copy Markdown
Contributor

It is very likely that the current package version for this feedstock is out of date.

Checklist before merging this PR:

  • Dependencies have been updated if changed: see upstream
  • Tests have passed
  • Updated license if changed and license_file is packaged

Information about this PR:

  1. Feel free to push to the bot's branch to update this PR if needed.
  2. The bot will almost always only open one PR per version.
  3. The bot will stop issuing PRs if more than 3 version bump PRs generated by the bot are open. If you don't want to package a particular version please close the PR.
  4. If you want these PRs to be merged automatically, make an issue with @conda-forge-admin,please add bot automerge in the title and merge the resulting PR. This command will add our bot automerge feature to your feedstock.
  5. If this PR was opened in error or needs to be updated please add the bot-rerun label to this PR. The bot will close this PR and schedule another one. If you do not have permissions to add this label, you can use the phrase @conda-forge-admin, please rerun bot in a PR comment to have the conda-forge-admin add it for you.

Pending Dependency Version Updates

Here is a list of all the pending dependency version updates for this repo. Please double check all dependencies before merging.

Name Upstream Version Current Version
cudnn 9.12.0.46 Anaconda-Server Badge

This PR was created by the regro-cf-autotick-bot. The regro-cf-autotick-bot is a service to automatically track the dependency graph, migrate packages, and propose package version updates for conda-forge. Feel free to drop us a line if there are any issues! This PR was generated by https://github.com/regro/cf-scripts/actions/runs/17107413748 - please use this URL for debugging.

@conda-forge-admin

conda-forge-admin commented Aug 20, 2025

Copy link
Copy Markdown
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

I do have some suggestions for making it better though...

For recipe/meta.yaml:

  • ℹ️ The recipe is not parsable by parser conda-souschef (grayskull). This parser is not currently used by conda-forge, but may be in the future. We are collecting information to see which recipes are compatible with grayskull.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/17881474427. Examine the logs at this URL for more detail.

@lucascolley

Copy link
Copy Markdown
Member

Hunk #1 FAILED at 22.

sounds like patches need to be regenerated?

@h-vetinari
h-vetinari force-pushed the 0.7.1_h1f8390 branch 2 times, most recently from ad947a0 to 2d6913d Compare August 27, 2025 12:33
h-vetinari and others added 10 commits September 11, 2025 21:38
…5.09.12.21.16.01

Other tools:
- conda-build 25.7.0
- rattler-build 0.47.0
- rattler-build-conda-compat 1.4.6
…5.09.15.11.24.57

Other tools:
- conda-build 25.7.0
- rattler-build 0.47.0
- rattler-build-conda-compat 1.4.6
@xhochy

xhochy commented Sep 16, 2025

Copy link
Copy Markdown
Member

@h-vetinari May I ask you for a favour here? I fail at changing all compiler pins to clang 19. Could you have a look and help me?

…5.09.16.07.18.04

Other tools:
- conda-build 25.7.0
- rattler-build 0.47.0
- rattler-build-conda-compat 1.4.6
@h-vetinari

Copy link
Copy Markdown
Member

The switch to clang got lost somewhere in the bowels of bazel, which seems to be looking for GCC still...

@traversaro

Copy link
Copy Markdown
Contributor

The switch to clang got lost somewhere in the bowels of bazel, which seems to be looking for GCC still...

I am not sure if it is related (I would need to check more in details) but while I was debugging another issue I remember seeing jax-ml/jax#29672 . I wanted to warn about this here, but then I forgot.

…5.09.16.07.18.04

Other tools:
- conda-build 25.7.0
- rattler-build 0.47.0
- rattler-build-conda-compat 1.4.6
…5.09.16.07.18.04

Other tools:
- conda-build 25.7.0
- rattler-build 0.47.0
- rattler-build-conda-compat 1.4.6
@xhochy

xhochy commented Sep 16, 2025

Copy link
Copy Markdown
Member

The switch to clang got lost somewhere in the bowels of bazel, which seems to be looking for GCC still...

This depends on some changes in bazel-toolchain. I have them locally and will push them in the next days once everything runs smoothly.

@xhochy

xhochy commented Sep 16, 2025

Copy link
Copy Markdown
Member

The switch to clang got lost somewhere in the bowels of bazel, which seems to be looking for GCC still...

I am not sure if it is related (I would need to check more in details) but while I was debugging another issue I remember seeing jax-ml/jax#29672 . I wanted to warn about this here, but then I forgot.

Good catch! The build is using the local toolchain for me but downloads a massive amount of LLVM binaries. So this is probably related.

@xhochy

xhochy commented Sep 18, 2025

Copy link
Copy Markdown
Member

I have a local fix for osx-arm64 (trivial one). I don't understand the osx-64 error. Would we be OK with skipping it for now? Then I would push my fix and build the full matrix.

@h-vetinari

Copy link
Copy Markdown
Member

I don't understand the osx-64 error.

It looks like a trivially missing header from onednn

In file included from external/xla/xla/service/cpu/runtime_matmul_c128.cc:22:
In file included from external/xla/xla/service/cpu/runtime_matmul_common.h:29:
external/xla/xla/tsl/framework/contraction/eigen_contraction_kernel.h:40:10: fatal error: 'dnnl.h' file not found
   40 | #include "dnnl.h"
      |          ^~~~~~~~
1 error generated.

@xhochy

xhochy commented Sep 18, 2025

Copy link
Copy Markdown
Member

The error goes a bit deeper. It seems that the onednn archive is not unpacked at all. As simple as the error looks, the solution is not obvious to me.

@xhochy

xhochy commented Sep 18, 2025

Copy link
Copy Markdown
Member

I have a script to bisect that in the XLA repository. This will take a while, but only CPU, not human, time.

@xhochy

xhochy commented Sep 19, 2025

Copy link
Copy Markdown
Member
fe13dfb8ed08f5152a9e65f03bfe2f398e1d2927 is the first bad commit
commit fe13dfb8ed08f5152a9e65f03bfe2f398e1d2927
Author: Penporn Koanantakool <penporn@google.com>
Date:   Fri Aug 8 07:32:29 2025 -0700

    [xla:cpu] Add a unified `cc_library` target for oneDNN.

    The original `mkl_deps()` macro can't be used in a select statement because it also has a select statement inside. (Bazel doesn't support nested selects.)

    Users should depend on this new `cc_library` target instead.

    PiperOrigin-RevId: 792617157

 xla/backends/cpu/BUILD                | 15 ++-------------
 xla/backends/cpu/runtime/onednn/BUILD | 16 ++++------------
 xla/tsl/framework/contraction/BUILD   |  3 +--
 xla/tsl/mkl/BUILD.bazel               | 12 ++++++++----
 4 files changed, 15 insertions(+), 31 deletions(-)

@h-vetinari

h-vetinari commented Sep 20, 2025

Copy link
Copy Markdown
Member

Since the linux builds passed previously and e9b7b9f is osx-specific, I'm cancelling the linux builds, due to heavy congestion on the opengpu server.

…5.09.20.07.44.53

Other tools:
- conda-build 25.7.0
- rattler-build 0.47.0
- rattler-build-conda-compat 1.4.6

@h-vetinari h-vetinari left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks so much for the huge effort here @xhochy! ❤️

@h-vetinari

Copy link
Copy Markdown
Member

Since the OSX failures were just a question of timeouts where we'll have to play with restarts a bit (and we have some server capacity right now), I'm going to merge this as-is.

@h-vetinari
h-vetinari merged commit 00dd066 into conda-forge:main Sep 21, 2025
23 of 28 checks passed
@regro-cf-autotick-bot
regro-cf-autotick-bot deleted the 0.7.1_h1f8390 branch September 21, 2025 06:48
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.

6 participants