min-guk opened a new pull request, #2104: URL: https://github.com/apache/systemds/pull/2104
This PR separates part of the content from a previous PR (https://github.com/apache/systemds/pull/2103). - `TestUtils::writeTestMatrix()`: When writing mtx files, the last number in the header should represent the number of non-zero elements in the matrix. However, in the existing JavaTest, it was showing the total number of elements, which caused an error when reading the mtx file using Python's `scipy.mmread()` function. This bug has been fixed. (Reference: https://networkrepository.com/mtx-matrix-market-format.html) - `TestUtils::readRMatrixFromFS()`: While R writes the mtx header in 2 lines, Python's scipy writes it in 3 lines, including an additional line containing only a `%` symbol. I modified the code to skip such blank lines when reading mtx files in JavaTest, so that no errors occur. -- 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. To unsubscribe, e-mail: dev-unsubscr...@systemds.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
