cjolivier01 opened a new pull request #7935: Zeroes CSR still needs a valid row_pointer array. URL: https://github.com/apache/incubator-mxnet/pull/7935 Fix for: https://github.com/apache/incubator-mxnet/issues/7920 **Looks like problem was introduced here** https://github.com/apache/incubator-mxnet/commit/0b1363116c84dcefa751a925749b2da04c3f2614 **In function** void FillZerosCsrImpl(mshadow::Stream<xpu> *s, NDArray *dst) **Why?** CSR matrix, even if all "zeros", must have m + 1 items in its row pointer (csr::IndPtr) array. Current implementation leaves all aux arrays empty. While I was here, added a couple more assertions and also reduced memory allocation for situations where lhs and rhs are the same array. Added unit test. ---------------------------------------------------------------- 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
