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


   Hi,
   This PR adds support for federated **binary, aggregate ternary, aggregate 
binary, ctable, and mmchain** instructions with multiple **aligned** federated 
matrices.
   I introduced a new enum _AType_ and the method _isAligned(AType)_ inside the 
FederationMap to simplify the alignment checks (to check for multiple possible 
partial alignments with a single call).
   To avoid redundant code, I created a new static method _isFederated()_ in 
_SpoofFEDInstruction_, which is only a consolidation of the _isFederated()_ 
method of _SpoofCPInstruction_ and _SpoofSPInstruction_.
   In addition, I changed several tests in order to also test multiple 
federated inputs.
   
   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