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

Upgrade orbax deps? #32

Open
ihh opened this issue Sep 6, 2023 · 4 comments · May be fixed by #33
Open

Upgrade orbax deps? #32

ihh opened this issue Sep 6, 2023 · 4 comments · May be fixed by #33
Assignees

Comments

@ihh
Copy link

ihh commented Sep 6, 2023

Installing with pip3 fails with the following error

      *** Orbax is a namespace, and not a standalone package. For model checkpointing and exporting utilities, please install `orbax-checkpoint` and `orbax-export` respectively (instead of `orbax`). ***
@alvarobartt
Copy link
Owner

alvarobartt commented Sep 7, 2023

Hi @ihh thanks for reporting this issue! I'm happy to go ahead with the upgrade of orbax and/or add orbax-export, would you like to contribute with a PR for that? 🤗

@ihh
Copy link
Author

ihh commented Sep 7, 2023

Thanks @alvarobartt. I've given it a try (see incoming PR), but unfortunately I am still unable to use safejax myself (or to test it) since I am encountering many dependency conflicts beyond just this issue. Many of these may be because of my development environment (I am working on a Mac) or my own naivete (I'm new to Python) but I am posting the PR anyway as a starting point for discussion.

@VolodyaCO
Copy link

Same here. The required jax versions are very old.

@alvarobartt
Copy link
Owner

Hi here @VolodyaCO I'm glad that someone's using safejax (or at least trying to) 🤗

Could you please let me know which are the ideal dependency versions you'd like to see pinned for JAX, Flax, etc.? Thanks in advance!

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 a pull request may close this issue.

3 participants