Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added load partioning for distributed runtime. #489

Merged
merged 1 commit into from
Apr 25, 2023

Conversation

aristotelis96
Copy link
Collaborator

Abstracted duplicate code for different distributed runtime communication frameworks.
Added distributed/coordinator/scheduling/LoadPartitioningDistributed.h class which is used to:

  • Split ranges based on some policy. For now only broadcast and simple distribution is supported.
  • Sets the metadata of the matrix being distributed/broadcasted.
  • Contains a static method which sets the metadata of the outputs during the computation step.
  • This class can be extended in order to support more complex distribution strategies.

Abstracted duplicate code for different distributed runtime communication frameworks.
@aristotelis96 aristotelis96 merged commit 54db063 into main Apr 25, 2023
@aristotelis96 aristotelis96 deleted the load-partitioning-distributed branch April 25, 2023 14:30
@corepointer
Copy link
Collaborator

Thanks for your contribution @aristotelis96 ! In the future, please add a reference to the issue/PR number to the first line of the commit message (e.g., "[DAPHNE-#489] Added load partitioning for distributed runtime"). That helps to reference any discussion that might relate to the change. Furthermore, if you merge/push from your local copy, then adding "Closes #489" in the last line of the commit message automatically changes the status of the open issue/PR to closed or merged.
Best regards, Mark

@aristotelis96
Copy link
Collaborator Author

Thanks for the tip @corepointer! I will make sure to include PR/issue number in future merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants