-
Notifications
You must be signed in to change notification settings - Fork 772
{lib}[foss/2020a] TensorFlow v2.3.1 w/ Python 3.8.2 #11546
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}[foss/2020a] TensorFlow v2.3.1 w/ Python 3.8.2 #11546
Conversation
…-GCCcore-9.3.0.eb, hwloc-1.11.12-GCCcore-9.3.0.eb, JsonCpp-1.9.3-GCCcore-9.3.0.eb, nsync-1.24.0-GCCcore-9.3.0.eb, protobuf-python-3.10.0-foss-2020a-Python-3.8.2.eb, TensorFlow-2.3.1-foss-2020a-Python-3.8.2.eb
This comment has been minimized.
This comment has been minimized.
easybuild/easyconfigs/p/protobuf-python/protobuf-python-3.10.0-foss-2020a-Python-3.8.2.eb
Outdated
Show resolved
Hide resolved
| # Dependencies created and updated using findPythonDeps.sh: | ||
| # https://gist.github.com/Flamefire/49426e502cd8983757bd01a08a10ae0d | ||
| exts_list = [ | ||
| ('Markdown', '3.2.2', { |
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.
@terjekv These should be revisited, some may already included with Python, some should probably be updated?
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.
What I did: eb --dump-env on this EC, then source it and run the linked script. That should give you a nice list of packages and versions to use.
This comment has been minimized.
This comment has been minimized.
|
@terjekv you've added hwloc-1.11.12, but you're not actually using it in the TF easyconfig, is that on purpose? |
Bah, thanks. I moved the PR to use hwloc-2.2.0 and forgot to remove the old one. Fixed now. Thanks again! |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
Test report by @smoors |
Nice catch. Should be resolved in 8d8dbd6. |
|
Test report by @smoors |
smoors
left a comment
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
|
@boegelbot please test @ generoso |
|
@boegel: Request for testing this PR well received on generoso PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 714332997 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
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)... Details- notification for comment with ID 714379585 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @terjekv |
|
Test report by @branfosj |
|
@branfosj Hmm... => @Flamefire Any ideas? |
|
Test report by @boegelbot |
|
Test report by @boegel |
|
Test report by @boegel |
|
Test report by @boegel |
|
Test report by @branfosj |
|
@branfosj That last report looks like it's coming from the same system as the previous one that showed the error, do you know what fixed it? Just |
|
@ocaisa The reports are from different systems. The failed build (bear-pg0305u03a) is on RHEL 7 - with the dependencies built over the last few months. The successful build (bear-pg0306u19a) is on RHEL 8 - with all the dependencies built today. I'm currently trying selective rebuild of the dependencies to see if I can get it to build. |
|
Ok, @surak is seeing a similar error on our systems for centos7 I think |
|
Ok. In order I've:
With those TensorFlow builds for me on the system where it was failing before. I've set off a test report. I cannot go back and check, but it may just need ICU rebuilding. My guess for rebuilding ICU was hoping that the |
|
Test report by @branfosj |
smoors
left a comment
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 @terjekv! |
This was fixed by easybuilders/easybuild-framework#3401 |
|
Just to add for future reference: I hit exactly the same |
(created using
eb --new-pr)Requires
#11543