[Operator] Make sure the result of topk is sorted

[sxjscience]

Currently, the inner implementation of topk is to first sort and slice the first K elements, which will make the results (value/indices) sorted. However, this is not mentioned in the doc. We need to revise the doc + test accordingly.

[safrooze]

@sxjscience Are you going to send a PR for this fix? If not, I can take care of it.

[sxjscience]

I’ve submitted this PR #11031 Get Outlook for iOS<https://aka.ms/o0ukef>

…

________________________________ From: Sina Afrooze <notifications@github.com> Sent: Tuesday, July 31, 2018 3:04:39 AM To: apache/incubator-mxnet Cc: Xingjian SHI; Mention Subject: Re: [apache/incubator-mxnet] [Operator] Make sure the result of topk is sorted (#10204) @sxjscience<https://github.com/sxjscience> Are you going to send a PR for this fix? If not, I can take care of it. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#10204 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AE8D7o2XSwNn-6kWaPz2d-yxE4pHVeQyks5uL1jHgaJpZM4S3hly>.

[sxjscience]

I mean #11134 Get Outlook for iOS<https://aka.ms/o0ukef>

…

________________________________ From: Xingjian SHI Sent: Tuesday, July 31, 2018 3:20:02 AM To: apache/incubator-mxnet; apache/incubator-mxnet Cc: Mention Subject: Re: [apache/incubator-mxnet] [Operator] Make sure the result of topk is sorted (#10204) I’ve submitted this PR #11031 Get Outlook for iOS<https://aka.ms/o0ukef>

________________________________ From: Sina Afrooze <notifications@github.com> Sent: Tuesday, July 31, 2018 3:04:39 AM To: apache/incubator-mxnet Cc: Xingjian SHI; Mention Subject: Re: [apache/incubator-mxnet] [Operator] Make sure the result of topk is sorted (#10204) @sxjscience<https://github.com/sxjscience> Are you going to send a PR for this fix? If not, I can take care of it. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#10204 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AE8D7o2XSwNn-6kWaPz2d-yxE4pHVeQyks5uL1jHgaJpZM4S3hly>.

[safrooze]

@sxjscience I don't see where in PR #11134 you update the doc for topk to mention that the result is sorted. Did I miss it in the diff or are you planning to add it later?

[thomelane]

See that it's now been added to #12250.
@sxjscience so looks like we're good to close this issue now then?

[sxjscience]

@thomelane Yes, it can be closed once #12250 is merged