zheng-da commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r181830262
########## File path: ci/docker/runtime_functions.sh ########## @@ -407,26 +405,28 @@ unittest_ubuntu_python3_gpu() { # quantization gpu currently only runs on P3 instances # need to separte it from unittest_ubuntu_python2_gpu() -unittest_ubuntu_python2_quantization_gpu() { +unittest_ubuntu_python2_quantization() { set -ex export PYTHONPATH=./python/ # MXNET_MKLDNN_DEBUG is buggy and produces false positives # https://github.com/apache/incubator-mxnet/issues/10026 #export MXNET_MKLDNN_DEBUG=1 # Ignored if not present export MXNET_STORAGE_FALLBACK_LOG_VERBOSE=0 nosetests-2.7 --verbose tests/python/quantization_gpu + nosetests-2.7 --verbose tests/python/quantization/test_quantization_mkldnn.py } # quantization gpu currently only runs on P3 instances # need to separte it from unittest_ubuntu_python3_gpu() -unittest_ubuntu_python3_quantization_gpu() { +unittest_ubuntu_python3_quantization() { Review comment: do we really have to test python2 and python3 in this case? @marcoabreu ---------------------------------------------------------------- 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