Yes, I think we need to add the doc to encourage the user to use int32. Get Outlook for iOS<https://aka.ms/o0ukef> ________________________________ From: Leonard Lausen <[email protected]> Sent: Tuesday, August 21, 2018 11:34:13 PM To: apache/incubator-mxnet Cc: Xingjian SHI; Mention Subject: Re: [apache/incubator-mxnet] [MXNET-507] Set arbitrary dtype for ret_indices in ordering ops (#12250)
@sxjscience<https://github.com/sxjscience> would it make sense to point out in the documentation that not specifying integer dtype may lead to the issues described in #11134<https://github.com/apache/incubator-mxnet/pull/11134> — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<https://github.com/apache/incubator-mxnet/pull/12250#issuecomment-414717864>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AE8D7pK4m_elT2HVhUu0POIurUaqJ6ESks5uTCh1gaJpZM4WEDe8>. [ Full content available at: https://github.com/apache/incubator-mxnet/pull/12250 ] This message was relayed via gitbox.apache.org for [email protected]
