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

Fixes flaky test_gluon_rnn.py:test_layer_bidirectional #14682

Merged
merged 1 commit into from
Apr 12, 2019

Conversation

perdasilva
Copy link
Contributor

Description

Fixes flaky test_gluon_rnn.py:test_layer_bidirectional. See #13103

@perdasilva
Copy link
Contributor Author

perdasilva commented Apr 12, 2019

Please wait for @szha ok before merging. These changes could impact on going RNN changes.

@@ -23,7 +23,7 @@
import unittest
from mxnet.test_utils import almost_equal, assert_almost_equal
from common import assert_raises_cudnn_not_satisfied

from tests.python.unittest.common import with_seed
Copy link
Member

Choose a reason for hiding this comment

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

Can we merge this line into L25?

from common import assert_raises_cudnn_not_satisfied, with_seed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

bad oversight on my behalf. Thank you for catching it!

@perdasilva perdasilva force-pushed the fix_flaky_test_layer_bidirectional branch from 236c9c5 to 27fede6 Compare April 12, 2019 13:49
@szha szha merged commit c788804 into apache:master Apr 12, 2019
larroy pushed a commit to larroy/mxnet that referenced this pull request Apr 15, 2019
@perdasilva perdasilva deleted the fix_flaky_test_layer_bidirectional branch May 7, 2019 16:33
perdasilva added a commit to perdasilva/incubator-mxnet that referenced this pull request May 7, 2019
haohuanw pushed a commit to haohuanw/incubator-mxnet that referenced this pull request Jun 23, 2019
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.

None yet

3 participants