Skip to content

fix: adapt to new jax API#338

Open
GaetanLepage wants to merge 1 commit intoborisdayma:mainfrom
GaetanLepage:jax-compat
Open

fix: adapt to new jax API#338
GaetanLepage wants to merge 1 commit intoborisdayma:mainfrom
GaetanLepage:jax-compat

Conversation

@GaetanLepage
Copy link

Tests are currently broken when ran with the latest version of JAX installed.
This patch ensures the code compatibility with the current jax API.

@borisdayma
Copy link
Owner

borisdayma commented Nov 15, 2023

Thanks, that's all you needed to change?
I'm scared the training code will most likely be broken with latest version as well.

I pinned the versions to use here:

jax==0.3.25

@GaetanLepage
Copy link
Author

Thanks, that's all you needed to change?

Well at least, it is enough for all of the tests to pass.
For the context, I was updating dalle-mini to the latest version on nixpkgs.

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.

2 participants