[GitHub] sxjscience commented on issue #12250: [MXNET-507] Set arbitrary dtype for ret_indices in ordering ops

[GitBox <gi...@apache.org>]

sxjscience commented on issue #12250: [MXNET-507] Set arbitrary dtype for ret_indices in ordering ops
URL: https://github.com/apache/incubator-mxnet/pull/12250#issuecomment-414720594
 
 
   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 <no...@github.com>
   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>.
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services