[GitHub] [incubator-mxnet] leezu commented on issue #17331: [mxnet 2.0] [item 2.4] Turning on large tensor support by default

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

leezu commented on issue #17331:
URL: https://github.com/apache/incubator-mxnet/issues/17331#issuecomment-656788437


   > NOTE: currently openBLAS works correctly with smaller inputs(within range of int32) but will truncate parameters passed with higher values and hence will result in either SIGSEGV(mostly) or garbage values being found(will eventually cause SIGSEGV in a bigger script)
   
   I'm a little concerned that we don't have a correct integration of BLAS and Lapack. BLAS kernels and will get potential crashes or corrupt results. But I think @sandeep-krishnamurthy's point
   
   > Make openBLAS compatible with Large Tensor support and merge the PR for Enabling Large Tensor Support so that default PyPi users of MXNet can already benefit from the new capability. This will actually cover largest user base of MXNet.
   
   refers to fixing this? If so, I'm fine with the order of execution. Thank you @access2rohit for the hard work on this feature


----------------------------------------------------------------
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