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

Add EC ZCH to default sharder #1610

Closed
wants to merge 3 commits into from

Conversation

PaulZhang12
Copy link
Contributor

Summary: Added the EC ZCH module to the get_default_sharders method

Differential Revision: D52541735

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jan 4, 2024
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52541735

PaulZhang12 added a commit to PaulZhang12/torchrec that referenced this pull request Jan 4, 2024
Summary:

Added the EC ZCH module to the get_default_sharders method

Differential Revision: D52541735
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52541735

PaulZhang12 added a commit to PaulZhang12/torchrec that referenced this pull request Jan 4, 2024
Summary:

Added the EC ZCH module to the get_default_sharders method

Reviewed By: henrylhtsang

Differential Revision: D52541735
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52541735

PaulZhang12 added a commit to PaulZhang12/torchrec that referenced this pull request Jan 4, 2024
Summary:

Added the EC ZCH module to the get_default_sharders method

Reviewed By: henrylhtsang

Differential Revision: D52541735
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52541735

Summary:

Allow ZCH module to also use EC

Reviewed By: dstaay-fb

Differential Revision: D51872310
Summary:

Support EC with Sharded ZCH modules

Reviewed By: dstaay-fb

Differential Revision: D52094769
Summary:

Added the EC ZCH module to the get_default_sharders method

Reviewed By: henrylhtsang

Differential Revision: D52541735
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52541735

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants