Skip to content

Conversation

bddppq
Copy link
Contributor

@bddppq bddppq commented Oct 31, 2018

caffe2 docker images uses native system python, which requires sudo to do pip install.
In pytorch rocm Ci we use caffe2 docker image

caffe2 docker images uses native system python, which requires sudo to do pip install.
In pytorch rocm Ci we use caffe2 docker image
@bddppq
Copy link
Contributor Author

bddppq commented Oct 31, 2018

@bddppq
Copy link
Contributor Author

bddppq commented Oct 31, 2018

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bddppq is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@bddppq bddppq deleted the pip-user-install branch October 31, 2018 06:41
facebook-github-bot pushed a commit that referenced this pull request Oct 31, 2018
Summary:
Recent PR #13366 added --user to pip install breaks ppc64le testing when using test.sh. This fix makes it not be used for ppc64le builds/test as both ninja and hypothesis are already in ppc64le docker images.
Pull Request resolved: #13388

Differential Revision: D12870164

Pulled By: soumith

fbshipit-source-id: b66bafc06ad2c5116bb5ef5e4681cf9c776084aa
@ezyang ezyang added the merged label Jun 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants