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.

Sign up for GitHub

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

Jump to bottom

Modified RSTensor to use parallel execution #247

Merged
merged 4 commits into from Jun 24, 2021
Merged

Modified RSTensor to use parallel execution #247

merged 4 commits into from Jun 24, 2021

Conversation

rasswanth-s
Copy link
Contributor

@rasswanth-s rasswanth-s commented Jun 21, 2021 •
edited

Description

The PR modifies RSTensor reconstruction to use parallel execution.

How has this been tested?

The RSTensor tests added before.

Checklist

@rasswanth-s rasswanth-s added this to the Replicated Sharing Tensor milestone Jun 21, 2021
kamathhrishi
kamathhrishi approved these changes Jun 23, 2021
View reviewed changes
Copy link
Contributor

@kamathhrishi kamathhrishi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

gmuraru
gmuraru approved these changes Jun 23, 2021
View reviewed changes
Copy link
Member

@gmuraru gmuraru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kamathhrishi kamathhrishi merged commit e790dd6 into OpenMined:main Jun 24, 2021
@rasswanth-s rasswanth-s deleted the reconstruct_parallel branch June 24, 2021 11:10
@rasswanth-s rasswanth-s self-assigned this Aug 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kamathhrishi kamathhrishi kamathhrishi approved these changes

@gmuraru gmuraru gmuraru approved these changes

@LaRiffle LaRiffle Awaiting requested review from LaRiffle

@aanurraj aanurraj Awaiting requested review from aanurraj aanurraj is a code owner

Assignees

@rasswanth-s rasswanth-s

Labels
refactor
Projects
None yet
Milestone
Replicated Sharing Tensor
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@rasswanth-s @kamathhrishi @gmuraru