[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #16371: Added large tensor support and test for gather_nd

[GitBox <gi...@apache.org>]

ChaiBapchya commented on a change in pull request #16371: Added large tensor support and test for gather_nd
URL: https://github.com/apache/incubator-mxnet/pull/16371#discussion_r336155936
 
 

 ##########
 File path: tests/nightly/test_large_array.py
 ##########
 @@ -1212,6 +1212,15 @@ def test_full():
     assert a[-1][-1] == 3
 
 
+def test_gather():
+    arr = mx.nd.ones((LARGE_X, SMALL_Y))
 
 Review comment:
   We do need parenthesis
   ```
   mx.nd.ones(3,4)
   Traceback (most recent call last):
     File "<stdin>", line 1, in <module>
     File "/Users/bapac/workspace/transpose/incubator-mxnet/python/mxnet/ndarray/ndarray.py", line 3153, in ones
       return _internal._ones(shape=shape, ctx=ctx, dtype=dtype, **kwargs)
     File "<string>", line 36, in _ones
     File "/Users/bapac/workspace/transpose/incubator-mxnet/python/mxnet/_ctypes/ndarray.py", line 100, in _imperative_invoke
       ctypes.byref(out_stypes)))
     File "/Users/bapac/workspace/transpose/incubator-mxnet/python/mxnet/base.py", line 254, in check_call
       raise MXNetError(py_str(_LIB.MXGetLastError()))
   mxnet.base.MXNetError: [11:15:17] ../include/mxnet/./base.h:526: Invalid context string 4
   ````

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


With regards,
Apache Git Services