Skip to content

Skip TestConvolution.test_convolution_sync on ROCM which caused random segfaults#10179

Closed
bddppq wants to merge 3 commits intopytorch:masterfrom
bddppq:disable-hip-test_convolution_sync
Closed

Skip TestConvolution.test_convolution_sync on ROCM which caused random segfaults#10179
bddppq wants to merge 3 commits intopytorch:masterfrom
bddppq:disable-hip-test_convolution_sync

Conversation

@bddppq
Copy link
Contributor

@bddppq bddppq commented Aug 2, 2018

@bddppq bddppq requested review from Jorghi12, ezyang and houseroad August 2, 2018 21:14
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 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

engine=st.sampled_from(["CUDNN", ""]))
def test_convolution_sync(self, net_type, num_workers, do, engine):
engine=st.sampled_from(["CUDNN", ""]),
**hu.gcs_ho_hip)

This comment was marked as off-topic.

This comment was marked as off-topic.

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 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

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 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@bddppq
Copy link
Contributor Author

bddppq commented Aug 3, 2018

@pytorchbot retest this please

goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
…m segfaults (pytorch#10179)

Summary:
https://ci.pytorch.org/jenkins/job/caffe2-builds/job/py2-clang3.8-rocm1.7.1-ubuntu16.04-test/4701/console

petrex ashishfarmer rohithkrn
Pull Request resolved: pytorch#10179

Differential Revision: D9139657

Pulled By: bddppq

fbshipit-source-id: 9b1bb2ad185ed16fff696ce026a5ee5fcf9cbaee
@ezyang ezyang added the merged label Jun 26, 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