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

add custom easyblock for JAX #2262

Closed
wants to merge 15 commits into from
Closed

Conversation

edmondac
Copy link
Contributor

@edmondac edmondac commented Dec 2, 2020

@edmondac edmondac changed the title WIP: JAX easyblock JAX easyblock Dec 3, 2020
@edmondac
Copy link
Contributor Author

edmondac commented Dec 3, 2020

See easybuilders/easybuild-easyconfigs#11595 for successful test reports. The CI on there won't work until this PR is merged, I believe.

@Flamefire
Copy link
Contributor

@edmondac Could you factor out the Bazel EB? I'd like to see that merged fast as my TF stuff is blocked by that ****

@edmondac
Copy link
Contributor Author

edmondac commented Dec 7, 2020

@edmondac Could you factor out the Bazel EB? I'd like to see that merged fast as my TF stuff is blocked by that ****

Sure - Bazel commits cherry-picked to a new branch in #2272

@easybuilders easybuilders deleted a comment from boegelbot Dec 8, 2020
@boegel boegel added this to the 4.4.0 milestone Dec 8, 2020
@boegel boegel changed the title JAX easyblock add custom easyblock for JAX Dec 8, 2020
@boegel boegel added the new label Dec 8, 2020
@boegel boegel removed this from the 4.4.0 milestone May 26, 2021
@boegel boegel added this to the release after 4.4.0 milestone May 26, 2021
@branfosj
Copy link
Member

easybuilders/easybuild-easyconfigs#11595 is closed as we are going with a different method for installing jax / jaxlib, so this is unneeded.

@branfosj branfosj closed this Aug 12, 2021
@Flamefire
Copy link
Contributor

We might revisit this though depending on how hard the EC turns out to be and if we want to move stuff out of it. But technically we don't need an easyblock, well besides patching the TF dependency

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants