-
Notifications
You must be signed in to change notification settings - Fork 706
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
reorganize Flax/JAX stack in 2023a: move jax
+ Optax
to gfbf/2023a
toolchain + use standalone Flax
+ absl-py
as dependencies
#21038
reorganize Flax/JAX stack in 2023a: move jax
+ Optax
to gfbf/2023a
toolchain + use standalone Flax
+ absl-py
as dependencies
#21038
Conversation
@boegelbot: please test @ generoso |
@lexming: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2248003332 processed Message to humans: this is just bookkeeping information for me, |
Test report by @verdurin |
@boegelbot: please test @ generoso |
@boegel: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2260701967 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
This comment was marked as off-topic.
This comment was marked as off-topic.
@boegel Test in generoso failed due to a dangling lock
|
Test report by @lexming |
('flit_core', '3.9.0', { | ||
'checksums': ['72ad266176c4a3fcfab5f2930d76896059851240570ce9a98733b658cb786eba'], | ||
}), | ||
('absl-py', '2.1.0', { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lexming Why isn't absl-py` required as dependency if this is removed? Was that incorrect to begin with?
I've cleaned up the lock, trying again... |
@boegelbot please test @ generoso |
@boegel: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2317735713 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@boegelbot please test @ jsc-zen3 |
@boegel: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2318254795 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegel |
test report failed for me because the existing |
Test report by @boegelbot |
Test report by @boegel |
@boegel you test failed because your robots path has both |
Indeed. I'm a bit confused why this is not a problem for the bots though (since they use |
Because you have On generoso:
|
Exactly, I overlooked that part. You're 100% right. I'll dance around this using |
Test report by @boegel |
jax
+ Optax
to gfbf/2023a
toolchain + use standalone Flax
+ absl-py
as dependencies
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Going in, thanks @lexming! |
(created using
eb --new-pr
)Adding new easyconfig for
Flax
, which deserves its own package; and forabsl-py
, which is used in many places already.This clarifies the dependency tree of the Flax/JAX stack as:
absl-py
>jax
>Optax
>Flax
Changelog:
jax
v0.4.25 from foss/2023a to gfbf/2023a following the changes introduced in {tools}[gfbf/2023a] jax v0.4.25 w/ CUDA 12.1.1 #20119jax
v0.4.25:absl-py
with regular dependencysource_urls
individually per each source sownloadedOptax
v0.2.2 from foss/2023a to gfbf/2023a and remove redundant extensions already provided by its dependenciesFlax
v0.8.4scvi-tools
already provided by its dependencies