-
Notifications
You must be signed in to change notification settings - Fork 705
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 patches to fix TensorFlow 2.7.1 on POWER #16795
add patches to fix TensorFlow 2.7.1 on POWER #16795
Conversation
Test report by @Flamefire |
5ac6fb6
to
62a8e13
Compare
Test report by @Flamefire |
@boegel I'd really love to get this into the next release as it was a lot of time spent and this is now the latest version of TensorFlow working on PPC. 2.8.4 doesn't has a CUDA version and doesn't work on PPC and 2.9 isn't ready yet. Test report coming up, need to install to /tmp as our clusters filesystem got full hence the (now deleted) failed test reports. But I tested it manually already so I expect no failures. |
Test report by @Flamefire |
Test report by @Flamefire |
This comment was marked as outdated.
This comment was marked as outdated.
d589592
to
627cab0
Compare
627cab0
to
672f528
Compare
@boegel What is missing here that it gets postponed again and again? #16795 (comment) shows it is working on PPC which is the purpose of this PR and the patches only affect the PPC build. |
Test report by @branfosj |
--> This doesn't make sense for the non-cuda build. Some left-overs from another build or other temporary failure? Or space issue in /dev/shm building 2 TFs? |
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.
I've tested the CPU-only version again and it failed on a GPU node (fresh login to the node). I also have a build running on a CPU-only node and it has not had the same issue. Testing further, this fault also occurs without the changes in this PR, which is not a surprise. So, I'm happy to approve this.
Test report by @branfosj |
Going in, thanks @Flamefire! |
(created using
eb --new-pr
)