Zha0q1 commented on a change in pull request #19025:
URL: https://github.com/apache/incubator-mxnet/pull/19025#discussion_r478704114
##########
File path: tests/nightly/test_np_large_array.py
##########
@@ -696,16 +696,16 @@ def test_one_hot():
assert A.grad[0] == 0
@use_np
[email protected](reason='backward value broken on large tensor')
def test_pick():
A = np.zeros((INT_OVERFLOW, 2))
B = np.zeros((INT_OVERFLOW))
+ A[0, 0] = 3
Review comment:
Accessing the element actually works fine. But I totally agree with you
in that to be 100% confident, I can fix assignment operator first and then try
this test again by setting a value of large index to something and then picking
it
----------------------------------------------------------------
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:
[email protected]
