access2rohit commented on a change in pull request #17462: Updated
PartialSortSmallK for LT support
URL: https://github.com/apache/incubator-mxnet/pull/17462#discussion_r397320540
##########
File path: tests/nightly/test_large_array.py
##########
@@ -1253,6 +1253,21 @@ def check_topk():
l = nd.topk(b, k=1, axis=-1, dtype=np.int64, ret_typ="value")
assert l.sum() == np.sum(np.arange(0, SMALL_Y))
+ def check_topk_small():
Review comment:
Can you add this input use case here instead:
https://github.com/apache/incubator-mxnet/blob/master/tests/python/unittest/test_operator.py#L4414
This file is for Large Tensor test cases only.
----------------------------------------------------------------
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
