[GitHub] reminisce commented on issue #7935: Zeroes CSR still needs a valid row_pointer array.

2017-09-18 Thread git
reminisce commented on issue #7935: Zeroes CSR still needs a valid row_pointer array. URL: https://github.com/apache/incubator-mxnet/pull/7935#issuecomment-330423935 I agree that an all zero initialized `indptr` in a zero csr sounds more correct from the definition point of view of csr

[GitHub] reminisce commented on issue #7935: Zeroes CSR still needs a valid row_pointer array.

2017-09-18 Thread git
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