mboehm7 commented on PR #2254: URL: https://github.com/apache/systemds/pull/2254#issuecomment-2820501742
LGTM - thanks for the patch @aarnaty. During the merge I removed the unrelated boolean rewrite test (which also missed the license header and dml scripts), fixed the formatting (tabs over spaces in java tests), removed unnecessary imports, added checks for applied rewrites, and simplified the trace-transpose rewrite. -- 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
