asitstands commented on issue #11031: Use dtype=int for the indices returned by TopK URL: https://github.com/apache/incubator-mxnet/issues/11031#issuecomment-392798800 I think that `random.multinomial` also need to be in the list. I'm positive on the introducion of the new name `idx_dtype`, but using the old `dtype` also looks not bad for the consistency, as the operators in `random` package use `dtype` to specify the data type of the output array. We need to set the backward compatible default currently, but I agree on changing it as integer in 2.0. Anyway an index is an integer by its nature.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
