reminisce commented on a change in pull request #13143: [MXNET-1206] Support
NDArray indexing with None and Ellipsis
URL: https://github.com/apache/incubator-mxnet/pull/13143#discussion_r298899907
##########
File path: tests/python/unittest/test_operator.py
##########
@@ -4388,13 +4388,14 @@ def test_normal_case():
def test_empty_tensor():
shape = (2, 3, 0, 4)
- a = np.array([], dtype=np.int32).reshape(shape)
- b = mx.nd.array(a, ctx=default_context(), dtype=a.dtype)
- reps = (2, 4, 6)
+ with mx.np_compat():
Review comment:
Empty tensors are not supported anyway without numpy shape semantics. The
original test put here was actually not desirable. I'm fine with your change
here. It's just a matter of code tidiness.
----------------------------------------------------------------
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]
With regards,
Apache Git Services
