Skip to content

Conversation

bddppq
Copy link
Contributor

@bddppq bddppq commented Apr 5, 2019

No description provided.

@bddppq bddppq requested a review from gchanan April 5, 2019 20:39
@soumith
Copy link
Member

soumith commented Apr 5, 2019

can we leave some builds out. ROCm build often is in flaky state, and we want to make sure non-MKL-DNN path is also always working.

@bddppq
Copy link
Contributor Author

bddppq commented Apr 5, 2019

@soumith Which build variants do you suggest we to leave them out?
(note this is already not turned on in conda CI builds)

@bddppq
Copy link
Contributor Author

bddppq commented Apr 5, 2019

@pytorchbot retest this please

@soumith
Copy link
Member

soumith commented Apr 5, 2019

just pick one or two, your choice.

Copy link
Contributor

@gchanan gchanan left a comment

Choose a reason for hiding this comment

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

approved once one or two builds are left out :).

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 bddppq changed the title Turn on mkldnn in all builds except rocm Turn on mkldnn in most builds except rocm Apr 8, 2019
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 Apr 8, 2019

#17748

@facebook-github-bot
Copy link
Contributor

@bddppq merged this pull request in 239de16.

@bddppq bddppq deleted the mkldnn-ci branch April 8, 2019 21:22
zhangguanheng66 pushed a commit to zhangguanheng66/pytorch that referenced this pull request May 6, 2019
Summary: Pull Request resolved: pytorch#18965

Differential Revision: D14836931

Pulled By: bddppq

fbshipit-source-id: 463a9bc5043a1f3194158f7bbfae3b71c6cd4b20
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