smelihportakal opened a new pull request, #2460:
URL: https://github.com/apache/systemds/pull/2460

   This PR implements the rewrites requested in 
[SYSTEMDS-3941](https://issues.apache.org/jira/browse/SYSTEMDS-3941).
   
   Changes:
   - Add a static rewrite for `rowSums(a*A) => a*rowSums(A)`
   - Add a static rewrite for `colSums(a*A) => a*colSums(A)`
   - Add a dynamic rewrite for `sum(matrix(a, rows=b, cols=c)) => a*b*c`
   - Add tests for all new rewrites under 
`src/test/java/org/apache/sysds/test/functions/rewrite`
   - Adjust `RewriteFusedRandTest` to avoid interference from the new `rowSums` 
rewrite 
   
   Validation:
   - Added rowSums rewrite tests with rewrites enabled and disabled
   - Added colSums rewrite tests with rewrites enabled and disabled
   - Added constant matrix sum rewrite tests with rewrites enabled and disabled
   - Updated `RewriteFusedRandTest` so it continues to test the intended fused 
random rewrite behavior


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to