marcoabreu commented on a change in pull request #9791: CI test randomness 3
URL: https://github.com/apache/incubator-mxnet/pull/9791#discussion_r168411398
##########
File path: tests/python/unittest/test_gluon.py
##########
@@ -530,13 +562,19 @@ def __init__(self, **kwargs):
model.collect_params()
assert len(w) == 0
+
+@with_seed()
def test_sequential_warning():
with warnings.catch_warnings(record=True) as w:
+ # The following line permits the test to pass if run multiple times
+ warnings.simplefilter('always')
Review comment:
I think this sounds like a valid failure that should be resolved. Imagine a
user calling the same method in a server and failing after some time because we
might have some memory leak or something else that causes this error.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
