apeforest commented on issue #15360: Revert default return type for indices in argsort() and topk() to fp32 URL: https://github.com/apache/incubator-mxnet/pull/15360#issuecomment-505983642 @marcoabreu Yes, the breaking change is mainly that the default behavior (default return data type changed from float to integer), which is **more** correct given that the operator returns the index to an element in the tensor. However, it may break existing script when downstream operators expects a float output from argsort/topk. This was identified in one of the tests of Java language binding. @frankfliu. Nonetheless, we should make this breaking change in MXNet 2.0 but not before that.
---------------------------------------------------------------- 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
