From 7145250f335f38c6648a8fd2ca8cec302fd81ca2 Mon Sep 17 00:00:00 2001 From: Chaitanya Prakash Bapat Date: Tue, 28 Apr 2020 20:23:57 -0700 Subject: [PATCH] Fix nightly large_vector test caused by incorrect with_seed path (#18178) --- tests/nightly/test_large_array.py | 2 +- tests/nightly/test_large_vector.py | 2 +- tests/python/unittest/common.py | 21 --------------------- 3 files changed, 2 insertions(+), 23 deletions(-) diff --git a/tests/nightly/test_large_array.py b/tests/nightly/test_large_array.py index 0f0b373409b9..67efc45470b0 100644 --- a/tests/nightly/test_large_array.py +++ b/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 +from common import with_seed import unittest # dimension constants diff --git a/tests/nightly/test_large_vector.py b/tests/nightly/test_large_vector.py index 57fecaafc0ac..dafbc1e01e3a 100644 --- a/tests/nightly/test_large_vector.py +++ b/tests/nightly/test_large_vector.py @@ -27,7 +27,7 @@ from mxnet.test_utils import rand_ndarray, assert_almost_equal, rand_coord_2d, create_vector from mxnet import gluon, nd -from tests.python.unittest.common import with_seed +from common import with_seed import unittest # dimension constants diff --git a/tests/python/unittest/common.py b/tests/python/unittest/common.py index eac0265fb747..ca88430e5b21 100644 --- a/tests/python/unittest/common.py +++ b/tests/python/unittest/common.py @@ -310,27 +310,6 @@ def teardown_module(): mx.nd.waitall() -def with_post_test_cleanup(): - """ - Helper function that cleans up memory by releasing it from memory pool - Required especially by large tensor tests that have memory footprints in GBs. - """ - def test_helper(orig_test): - @functools.wraps(orig_test) - def test_new(*args, **kwargs): - logger = default_logger() - try: - orig_test(*args, **kwargs) - except: - logger.info(test_msg) - raise - finally: - mx.nd.waitall() - mx.cpu().empty_cache() - return test_new - return test_helper - - def run_in_spawned_process(func, env, *args): """ Helper function to run a test in its own process.