reminisce commented on issue #7935: Zeroes CSR still needs a valid row_pointer 
array.
URL: https://github.com/apache/incubator-mxnet/pull/7935#issuecomment-330371832
 
 
   I think it's okay for operators to work on a zero csr with empty `indptr` as 
long as the csr is checked if zero before performing any regular operations 
like the one exposed in https://github.com/apache/incubator-mxnet/issues/7920. 
In addition to what @eric-haibin-lin said about the intention of delaying 
memory allocation, I think it's also the intended way to avoid unnecessary 
memory allocation for zero csr tensors. We can provide a util function checking 
whether a sparse tensor (csr or rsp) is zero for being used in operator 
implementation.
 
----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to