haojin2 commented on a change in pull request #17657: Add numpy_fallback 
decorator
URL: https://github.com/apache/incubator-mxnet/pull/17657#discussion_r382876918
 
 

 ##########
 File path: python/mxnet/ndarray/ndarray.py
 ##########
 @@ -5087,7 +5087,7 @@ def _make_dl_managed_tensor(array):
         return array(ndarray, dtype=ndarray.dtype)
 
     if not ndarray.flags['C_CONTIGUOUS']:
-        raise ValueError("Only c-contiguous arrays are supported for 
zero-copy")
+        return array(ndarray, dtype=ndarray.dtype)
 
 Review comment:
   The same thing could actually be done in another way, I can catch this error 
outside the function and handle accordingly.

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to