You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@systemds.apache.org by GitBox <gi...@apache.org> on 2022/05/09 23:18:51 UTC
[GitHub] [systemds] OlgaOvcharenko opened a new pull request, #1609: [WIP][SYSTEMDS-3374] Federated primitive for transferring a local data object to a federated representation
OlgaOvcharenko opened a new pull request, #1609:
URL: https://github.com/apache/systemds/pull/1609
This PR adds a federated primitive for transferring a local data object (matrices and frames) to a federated representation, e.g. `federated(objects = list(matrix(0, rows=3, cols=4), matrix(0, rows=3, cols=4)), addresses=list($in_X1, $in_X2), ftype="ROW", type="MATRIX")`.
--
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-unsubscribe@systemds.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [systemds] asfgit closed pull request #1609: [SYSTEMDS-3374] Federated primitive for transferring a local data object to a federated representation
Posted by GitBox <gi...@apache.org>.
asfgit closed pull request #1609: [SYSTEMDS-3374] Federated primitive for transferring a local data object to a federated representation
URL: https://github.com/apache/systemds/pull/1609
--
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-unsubscribe@systemds.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [systemds] mboehm7 commented on pull request #1609: [SYSTEMDS-3374] Federated primitive for transferring a local data object to a federated representation
Posted by GitBox <gi...@apache.org>.
mboehm7 commented on PR #1609:
URL: https://github.com/apache/systemds/pull/1609#issuecomment-1146840484
LGTM - thanks for the patch @OlgaOvcharenko. I will merge this in with minor changes as a number of merge conflicts already accumulated. However, there is quite some redundancy in `InitFEDInstruction` - could you please (in a follow-up PR) eliminate this redundancy, construct a federatedMap, and subsequently simply call broadcastSliced as an existing primitive for transferring slices of a local data object to individual federated workers. That way, we're reuse the well-tested and compact runtime primitive without need for changing the worker handler.
--
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-unsubscribe@systemds.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org