Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

[Do not merge] This PR (v2) is for check the fix in onDNN. #19260

Closed
wants to merge 2 commits into from

Conversation

pioy
Copy link

@pioy pioy commented Sep 30, 2020

This PR is for check the fix in onDNN (see #19251).
I additionally added a piece of code that make a lot of test failing without the fix.
If the tests passes then the fix works despite the code that introduce failing stimulus (which previously caused fails).

@mxnet-bot
Copy link

Hey @pioy , Thanks for submitting the PR
All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands:

  • To trigger all jobs: @mxnet-bot run ci [all]
  • To trigger specific jobs: @mxnet-bot run ci [job1, job2]

CI supported jobs: [sanity, centos-gpu, website, windows-cpu, unix-cpu, clang, centos-cpu, edge, miscellaneous, unix-gpu, windows-gpu]


Note:
Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin.
All CI tests must pass before the PR can be merged.

@pioy
Copy link
Author

pioy commented Sep 30, 2020

@mxnet-bot run ci centos-cpu

@mxnet-bot
Copy link

Undefined action detected.
Permissible actions are : run ci [all], run ci [job1, job2]
Example : @mxnet-bot run ci [all]
Example : @mxnet-bot run ci [centos-cpu, clang]

@pioy
Copy link
Author

pioy commented Sep 30, 2020

@mxnet-bot run ci [centos-cpu]

@pioy
Copy link
Author

pioy commented Sep 30, 2020

@mxnet-bot run ci [centos-cpu]

@mxnet-bot
Copy link

Jenkins CI successfully triggered : [centos-cpu]

@pioy
Copy link
Author

pioy commented Sep 30, 2020

@mxnet-bot run ci [unix-cpu]

@mxnet-bot
Copy link

Jenkins CI successfully triggered : [unix-cpu]

@pioy
Copy link
Author

pioy commented Sep 30, 2020

@mxnet-bot run ci [centos-cpu]

@mxnet-bot
Copy link

Jenkins CI successfully triggered : [centos-cpu]

@pioy pioy closed this Oct 1, 2020
@pioy
Copy link
Author

pioy commented Oct 1, 2020

It looks like I missed something switching to my onDNN repo which, in fact, did not happen.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants