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

Mark test_np_average as flaky #19538

Merged
merged 1 commit into from
Nov 18, 2020
Merged

Mark test_np_average as flaky #19538

merged 1 commit into from
Nov 18, 2020

Conversation

leezu
Copy link
Contributor

@leezu leezu commented Nov 16, 2020

[2020-11-14T04:52:26.695Z] if returned:
[2020-11-14T04:52:26.695Z] np_out, np_sum_of_weights = np_out
[2020-11-14T04:52:26.695Z] mx_out, mx_sum_of_weights = mx_out
[2020-11-14T04:52:26.695Z] assert_almost_equal(mx_sum_of_weights.asnumpy(), np_sum_of_weights, rtol=rtol, atol=atol)
[2020-11-14T04:52:26.695Z] assert mx_out.shape == np_out.shape
[2020-11-14T04:52:26.695Z] > assert_almost_equal(mx_out.asnumpy(), np_out, rtol=rtol, atol=atol)
[...]
[2020-11-14T04:52:26.695Z] > raise AssertionError(msg)
[2020-11-14T04:52:26.695Z] E AssertionError:
[2020-11-14T04:52:26.695Z] E Items are not equal:
[2020-11-14T04:52:26.695Z] E Error 1.772945 exceeds tolerance rtol=1.000000e-03, atol=1.000000e-04.
[2020-11-14T04:52:26.695Z] E
[2020-11-14T04:52:26.695Z] E ACTUAL: array(39093.145, dtype=float32)
[2020-11-14T04:52:26.695Z] E DESIRED: 39023.957

[2020-11-14T04:52:26.695Z]         if returned:
[2020-11-14T04:52:26.695Z]             np_out, np_sum_of_weights = np_out
[2020-11-14T04:52:26.695Z]             mx_out, mx_sum_of_weights = mx_out
[2020-11-14T04:52:26.695Z]             assert_almost_equal(mx_sum_of_weights.asnumpy(), np_sum_of_weights, rtol=rtol, atol=atol)
[2020-11-14T04:52:26.695Z]         assert mx_out.shape == np_out.shape
[2020-11-14T04:52:26.695Z] >       assert_almost_equal(mx_out.asnumpy(), np_out, rtol=rtol, atol=atol)
[...]
[2020-11-14T04:52:26.695Z] >       raise AssertionError(msg)
[2020-11-14T04:52:26.695Z] E       AssertionError:
[2020-11-14T04:52:26.695Z] E       Items are not equal:
[2020-11-14T04:52:26.695Z] E       Error 1.772945 exceeds tolerance rtol=1.000000e-03, atol=1.000000e-04.
[2020-11-14T04:52:26.695Z] E
[2020-11-14T04:52:26.695Z] E        ACTUAL: array(39093.145, dtype=float32)
[2020-11-14T04:52:26.695Z] E        DESIRED: 39023.957
@mxnet-bot
Copy link

Hey @leezu , 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: [centos-cpu, centos-gpu, edge, unix-cpu, unix-gpu, windows-gpu, website, windows-cpu, sanity, miscellaneous, clang]


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.

@lanking520 lanking520 added the pr-awaiting-testing PR is reviewed and waiting CI build and test label Nov 16, 2020
@leezu
Copy link
Contributor Author

leezu commented Nov 17, 2020

@mxnet-bot run ci [windows-gpu]

@mxnet-bot
Copy link

Jenkins CI successfully triggered : [windows-gpu]

@lanking520 lanking520 added pr-awaiting-merge Review and CI is complete. Ready to Merge and removed pr-awaiting-testing PR is reviewed and waiting CI build and test labels Nov 18, 2020
@leezu leezu merged commit f998ac6 into apache:master Nov 18, 2020
@leezu leezu deleted the flaky branch November 18, 2020 02:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr-awaiting-merge Review and CI is complete. Ready to Merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants