-
Notifications
You must be signed in to change notification settings - Fork 284
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
Fix $PYTHONPATH
for hermetic python in TensorFlow builds
#3568
Conversation
While testing this I also realized that the Bringing that patch back does not seem to trivially fix our |
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.
well, lgtm, it only impacts the build step, so it seems safe.
@yqshao if you could upload a test report to this i would appreciate it |
Test report by @yqshao Overview of tested easyconfigs (in order)
Build succeeded for 16 out of 17 (1 easyconfigs in total) |
^ above is an example of the failure with |
Test report by @yqshao Overview of tested easyconfigs (in order)
Build succeeded for 1 out of 1 (1 easyconfigs in total) |
$PYTHONPATH
for hermetic python in TensorFlow builds
Trying to fix #3566