You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mxnet.apache.org by gi...@git.apache.org on 2017/10/13 17:40:17 UTC

[GitHub] piiswrong commented on a change in pull request #8244: [WIP] generate op frontend code for IDE auto-complete

piiswrong commented on a change in pull request #8244: [WIP] generate op frontend code for IDE auto-complete
URL: https://github.com/apache/incubator-mxnet/pull/8244#discussion_r144615586
 
 

 ##########
 File path: python/mxnet/ndarray/op.py
 ##########
 @@ -15,160 +15,22 @@
 # specific language governing permissions and limitations
 # under the License.
 
-"""Register backend ops in mxnet.ndarray namespace"""
+# coding: utf-8
+# pylint: disable=unused-import
+"""Backend ops in mxnet.ndarray namespace"""
 __all__ = ['CachedOp']
 
 import sys as _sys
 import os as _os
-import ctypes
-import numpy as np  # pylint: disable=unused-import
-
-from ..ndarray_doc import _build_doc
 
-# Use different version of SymbolBase
-# When possible, use cython to speedup part of computation.
-# pylint: disable=unused-import
 try:
     if int(_os.environ.get("MXNET_ENABLE_CYTHON", True)) == 0:
-        from .._ctypes.ndarray import NDArrayBase
-        from .._ctypes.ndarray import CachedOp, _imperative_invoke
+        from .._ctypes.ndarray import NDArrayBase, CachedOp
 
 Review comment:
   Import these from register?
 
----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services