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

Remove importing GlobalWeightDecayDefinition in split_table_batched_embeddings_ops #2904

Closed
wants to merge 1 commit into from

Conversation

spcyppt
Copy link
Contributor

@spcyppt spcyppt commented Jul 26, 2024

Summary: f585120721 packaged split_table_batched_embeddings_ops.py but not split_table_batched_embeddings_ops_training.py. Thus it fails to import GlobalWeightDecayDefinition from an older version of split_table_batched_embeddings_ops_training.py.

Reviewed By: spcyppt

Differential Revision: D60271207

…mbeddings_ops

Summary: f585120721 packaged `split_table_batched_embeddings_ops.py` but not `split_table_batched_embeddings_ops_training.py`. Thus it fails to import `GlobalWeightDecayDefinition` from an older version of `split_table_batched_embeddings_ops_training.py`.

Reviewed By: spcyppt

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

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

Copy link

netlify bot commented Jul 26, 2024

Deploy Preview for pytorch-fbgemm-docs ready!

Name Link
🔨 Latest commit 0397b28
🔍 Latest deploy log https://app.netlify.com/sites/pytorch-fbgemm-docs/deploys/66a30d96e85eb400083e7125
😎 Deploy Preview https://deploy-preview-2904--pytorch-fbgemm-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 2808977.

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

Successfully merging this pull request may close these issues.

2 participants