-
Notifications
You must be signed in to change notification settings - Fork 762
{tools}[foss/2020b] jax v0.2.19 #13622
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
{tools}[foss/2020b] jax v0.2.19 #13622
Conversation
|
Need another 2 changes in build.py:
|
Co-authored-by: Alexander Grund <[email protected]>
Co-authored-by: Alexander Grund <[email protected]>
Co-authored-by: Alexander Grund <[email protected]>
Co-authored-by: Alexander Grund <[email protected]>
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.
@deniskristak The name of the patch files as they are included in the PR are incorrect.
I fixed that in deniskristak#3, along with a couple of other things, like including checksums.
…lding jaxlib in parallel, use pip to install all Python packages, enable sanity_pip_check, fix and rename patch file for TensorFlow sources
…e of -march=native)
…ak/easybuild-easyconfigs into 20210805110301_new_pr_jax0218
|
Test report by @boegel edit: hmm, one of the tests failed... |
|
@boegelbot please test @ generoso |
|
@boegel: Request for testing this PR well received on generoso PR test command '
Test results coming soon (I hope)... - notification for comment with ID 898704698 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegel |
|
Test report by @boegel |
|
Test report by @boegel |
|
Test report by @boegelbot |
|
This can/should probably be cleaned up by implementing a custom easyblock for I'll merge this as is though, since it's working, and a major hurdle towards AlphaFold, and keep that as an exercise for an upcoming PR... |
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 @deniskristak! |
|
Test report by @Flamefire |
|
Test report by @Flamefire |
edit (by @boegel): requires
easybuilders/easybuild-easyblocks#2534