-
Notifications
You must be signed in to change notification settings - Fork 701
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
update TensorFlow 2.1.0 easyconfigs to provide more dependencies via EasyBuild #11109
Conversation
@boegel Whats going on with the tests? I didn't add neither hwloc nor ICU |
fad4551
to
3b6d8ab
Compare
easybuild/easyconfigs/t/TensorFlow/TensorFlow-2.1.0-foss-2019b-Python-3.7.4.eb
Outdated
Show resolved
Hide resolved
easybuild/easyconfigs/t/TensorFlow/TensorFlow-2.1.0-foss-2019b-Python-3.7.4.eb
Outdated
Show resolved
Hide resolved
94b8c8a
to
1ede6f9
Compare
Test report by @Flamefire |
1ede6f9
to
c6ea62a
Compare
easybuild/easyconfigs/t/TensorFlow/TensorFlow-2.1.0-foss-2019b-Python-3.7.4.eb
Outdated
Show resolved
Hide resolved
64ad00a
to
99be36f
Compare
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.
- also stick to
hwloc
1.x infoss
easyconfig
easybuild/easyconfigs/t/TensorFlow/TensorFlow-2.1.0-foss-2019b-Python-3.7.4.eb
Outdated
Show resolved
Hide resolved
991c12f
to
098dadc
Compare
@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 687114655 processed Message to humans: this is just bookkeeping information for me, |
Test report by @Flamefire |
Test report by @boegel |
Test report by @boegel |
Test report by @boegel edit: failed because of old |
Test report by @Flamefire |
Test report by @boegelbot |
Test report by @boegel edit: ignore this one, not enough memory available ( |
Test report by @boegel |
Test report by @boegel edit: tested on top of easybuilders/easybuild-easyblocks#2163 |
@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 687555263 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
Going in, thanks @Flamefire! |
Use pip to install protobuf-python in 2019b toolchain (requires re-downloading source tarball!) #11143(usepip
forprotobuf-python
){math}[fosscuda/2019b] scipy v1.4.1 w/ Python 3.7.4 #11216(scipy forfosscuda/2019b
){math}[foss/2019b] scipy v1.4.1 w/ Python 3.7.4 #11218(scipy 1.4.1 forfoss/2019b
)This is basically a test run. It makes the installation faster and smaller and avoids some downloads at build time.
This approach will be required for TF 2.3.0 and I'll add PRs for the other 2.x ECs once this is merged
Reason why it is required is the TF 2.3 bundles cURL but builds it with Bazel side-stepping the (correct) configuration from cURL and introducing a bug on some systems making parts of TF unusable: tensorflow/tensorflow#40065 (comment)