[GitHub] [incubator-mxnet] access2rohit commented on issue #16898: Sparse int64 Large tensor support

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

access2rohit commented on issue #16898: Sparse int64 Large tensor support
URL: https://github.com/apache/incubator-mxnet/pull/16898#issuecomment-569555239
 
 
   > @eric-haibin-lin while I don't think there is uniform guideline for int64 support across all ops. But in most cases, switching the datatype of all the variables holding "indices" from 32bit dtype to 64bit dtype solves the issue. One still has to test that to ensure it doesn't break the functionality and supports int64.
   > 
   > @apeforest @access2rohit any inputs?
   
   Plus ensure that the C API being used supports int64_t for indices as well. On language binding side int64_t values need to be passed to C++ backend. For e.g. ctype.int64 should be used for passing large values else they will get truncated at the language binding interface itself.

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


With regards,
Apache Git Services