ArmageddonKnight commented on a change in pull request #18704: URL: https://github.com/apache/incubator-mxnet/pull/18704#discussion_r488903487
########## File path: tests/python/gpu/test_profiler_gpu.py ########## @@ -15,44 +15,44 @@ # specific language governing permissions and limitations # under the License. +import csv import os import sys +import numpy as np import mxnet as mx mx.test_utils.set_default_context(mx.gpu(0)) curr_path = os.path.dirname(os.path.abspath(os.path.expanduser(__file__))) sys.path.insert(0, os.path.join(curr_path, '../unittest')) -# We import all tests from ../unittest/test_profiler.py -# They will be detected by test framework, as long as the current file has a different filename -from test_profiler import * +from mxnet import profiler +from mxnet.gluon import nn +from mxnet.gluon.block import _block_scope +from test_profiler import enable_profiler -# Test seen to crash pytest worker during development of https://github.com/apache/incubator-mxnet/pull/18694 -del test_aggregate_duplication Review comment: @leezu As we can see here, the test case was not working before. ---------------------------------------------------------------- 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: [email protected]
