ywcb00 opened a new pull request #1170:
URL: https://github.com/apache/systemds/pull/1170


   Hi,
   This is a PR for adding federated Als-CG as a first algorithm-level test for 
federated quaternary operations.
   I also added a branch inside FEDInstructionUtils for rewriting BinarySP 
instructions into BinaryFED instructions in order to support the "fed_!=" 
instruction needed inside the als-CG algorithm. Since I haven't found a test 
for these federated operations, I added a test for MatrixScalar and 
MatrixMatrix federated logical operations.
   I made a small fix (I hope it was a fix xD) inside 
BinaryMatrixMatrixFEDInstruction by changing a broadcast() to a 
broadcastSliced().
   Hope that was not too much, otherwise just revert the last commit (bce7fdc) 
and the AlsCG test is using "sp_!=" for spark execution mode again.
   
   Thanks for review :)


----------------------------------------------------------------
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.

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


Reply via email to