You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2020/04/29 01:31:21 UTC

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #18178: Fix nightly large_vector test caused by incorrect with_seed path

access2rohit commented on a change in pull request #18178:
URL: https://github.com/apache/incubator-mxnet/pull/18178#discussion_r417020159



##########
File path: tests/nightly/test_large_array.py
##########
@@ -27,7 +27,7 @@
 
 from mxnet.test_utils import rand_ndarray, assert_almost_equal, rand_coord_2d, default_context, check_symbolic_forward, create_2d_tensor
 from mxnet import gluon, nd
-from common import with_seed, with_post_test_cleanup

Review comment:
       Why remove this? It attempts to free up memory after each test passes. Its good to keep it in order to avoid OOM on CI instances.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org