-
Notifications
You must be signed in to change notification settings - Fork 704
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
{lib}[fosscuda/2019b] TensorFlow v2.4.1 w/ Python 3.7.4 #11637
{lib}[fosscuda/2019b] TensorFlow v2.4.1 w/ Python 3.7.4 #11637
Conversation
…nd patches: TensorFlow-2.4.0_add-default-shell-env.patch, TensorFlow-2.4.0_add-protobuf-deps.patch, TensorFlow-2.4.0_downgrade-required-versions.patch, TensorFlow-2.4.0_fix-eigen-on-power.patch, TensorFlow-2.4.0_replace-exectools-to-tools.patch
d911285
to
b9af6d4
Compare
Test report by @Flamefire |
Test report by @Flamefire |
Just got a notification that TF 2.4.1 is released, seems like only to make it more compatible, so I'd update this PR if that works: https://github.com/tensorflow/tensorflow/releases/tag/v2.4.1 |
Test report by @Flamefire |
Test report by @Flamefire |
easybuild/easyconfigs/t/TensorFlow/TensorFlow-2.4.1-fosscuda-2019b-Python-3.7.4.eb
Show resolved
Hide resolved
f3424c7
to
5ee669b
Compare
5ee669b
to
229e342
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 778280781 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegel |
Test report by @Flamefire |
Test report by @boegelbot |
Test report by @Flamefire |
Test report by @boegel |
Test report by @branfosj |
Still several problematic tests, it seems... All this testing does show that easybuilders/easybuild-easyblocks#2312 is working as intended though, so I'll go ahead and merge that. |
@branfosj The error from your run is:
This could be fixed by enabling permissions, see tensorflow/tensorflow#35860 (comment) Maybe we should exclude that test? |
Yes, we should exclude that test. |
Done. Could you rerun the test? |
Test report by @branfosj |
Test report by @boegel |
@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 779390737 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegel |
Test report by @boegel |
Test report by @boegelbot |
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 @Flamefire! |
(created using
eb --new-pr
)set $TF_GPU_COUNT and $TF_TESTS_PER_GPU for TensorFlow tests easybuild-easyblocks#2292improve Bazel easyblock: add support for running tests, enable static linking, use build dir rather than tmpdir, verbose output easybuild-easyblocks#2285The new Bazel version is not strictly required but it does require a statically build Bazel and the easiest way to enforce this is to require a new version built with easybuilders/easybuild-easyblocks#2285
add support for skipping steps in Python packages installed as extension + print progress on individual steps for installing Python packages as extensions easybuild-easyblocks#2290could also be useful, especially for this as the tests take quite long (similar to pytorch)