mboehm7 commented on PR #2279: URL: https://github.com/apache/systemds/pull/2279#issuecomment-3118977986
LGTM - thanks for the patch @sandrathena. During the merge I fixed the scrambled dml-language-ref, added the missing license header, fixed the formatting of the tests (tabs over spaces, no single line methods, imports), and left a FIXME for the spark instruction (which should be integrated with CumulativeAggregateSPInstruction, UnaryMatrixCPInstruction, and CumulativeOffsetSPInstruction). -- 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
