small edits according to reviews (apache#151)

cjolivier01 · Aug 8, 2017 · f318c9d · f318c9d
1 parent 2dc7dc9
commit f318c9d
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 5 deletions.
diff --git a/example/sparse/get_data.py b/example/sparse/get_data.py
@@ -5,7 +5,7 @@
 
 def get_libsvm_data(data_dir, data_name, url, data_origin_name):
     if not os.path.isdir(data_dir):
-        os.system("mkdir " + data_dir)
+        os.mkdir(data_dir)
     os.chdir(data_dir)
     if (not os.path.exists(data_name)):
         import urllib

diff --git a/python/mxnet/ndarray/ndarray.py b/python/mxnet/ndarray/ndarray.py
@@ -26,7 +26,7 @@
 from . import broadcast_greater, broadcast_greater_equal, broadcast_lesser, broadcast_lesser_equal
 from . import zeros_like, slice
 
-__all__ = ["NDArray", "concatenate", "_DTYPE_NP_TO_MX", "_DTYPE_MX_TO_NP",                   \
+__all__ = ["NDArray", "concatenate", "_DTYPE_NP_TO_MX", "_DTYPE_MX_TO_NP", "_GRAD_REQ_MAP",  \
            "ones", "add", "arange", "divide", "equal", "full", "greater", "greater_equal",   \
            "imdecode", "lesser", "lesser_equal", "maximum", "minimum", "moveaxis",           \
            "multiply", "negative", "not_equal", "onehot_encode", "power", "subtract",        \

diff --git a/python/mxnet/symbol.py b/python/mxnet/symbol.py
@@ -21,7 +21,7 @@
 from .base import NDArrayHandle, ExecutorHandle, SymbolHandle, OpHandle
 from .base import check_call, MXNetError, NotImplementedForSymbol, _Null  # pylint: disable=unused-import
 from .context import Context
-from .ndarray.ndarray import NDArray, _DTYPE_NP_TO_MX, _DTYPE_MX_TO_NP, _GRAD_REQ_MAP
+from .ndarray import NDArray, _DTYPE_NP_TO_MX, _DTYPE_MX_TO_NP, _GRAD_REQ_MAP
 from .name import NameManager  # pylint: disable=unused-import
 from .ndarray.ndarray import _STORAGE_TYPE_STR_TO_ID
 from .ndarray.sparse_ndarray import _ndarray_cls

diff --git a/src/executor/graph_executor.h b/src/executor/graph_executor.h
@@ -19,8 +19,6 @@
 #include <vector>
 #include "./exec_pass.h"
 
-#define EXECUTOR_DEBUG 0
-
 namespace mxnet {
 
 // forward declaration