-
Notifications
You must be signed in to change notification settings - Fork 463
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 opcheck tests for fbgemm::split_embedding_codegen_forward_[un]weighted_cuda #2205
Conversation
✅ Deploy Preview for pytorch-fbgemm-docs canceled.
|
This pull request was exported from Phabricator. Differential Revision: D52067413 |
…ghted_cuda (#2205) Summary: Title. Also the split_embedding_codegen_forward_[un]weighted_cuda ops as PT2 compliant (and mark split_embedding_codegen_lookup_{} functions I may have missed). Differential Revision: D52067413
f40fd28
to
d7ecdd3
Compare
This pull request was exported from Phabricator. Differential Revision: D52067413 |
d7ecdd3
to
1881d90
Compare
…ghted_cuda (#2205) Summary: Title. Also the split_embedding_codegen_forward_[un]weighted_cuda ops as PT2 compliant (and mark split_embedding_codegen_lookup_{} functions I may have missed). Differential Revision: D52067413
This pull request was exported from Phabricator. Differential Revision: D52067413 |
1881d90
to
cbe5ac4
Compare
…ghted_cuda (#2205) Summary: Title. Also the split_embedding_codegen_forward_[un]weighted_cuda ops as PT2 compliant (and mark split_embedding_codegen_lookup_{} functions I may have missed). Differential Revision: D52067413
This pull request was exported from Phabricator. Differential Revision: D52067413 |
cbe5ac4
to
5c25e5a
Compare
…ghted_cuda (#2205) Summary: Title. Also the split_embedding_codegen_forward_[un]weighted_cuda ops as PT2 compliant (and mark split_embedding_codegen_lookup_{} functions I may have missed). Reviewed By: zou3519 Differential Revision: D52067413
This pull request was exported from Phabricator. Differential Revision: D52067413 |
1 similar comment
This pull request was exported from Phabricator. Differential Revision: D52067413 |
…ghted_cuda (#2205) Summary: Title. Also the split_embedding_codegen_forward_[un]weighted_cuda ops as PT2 compliant (and mark split_embedding_codegen_lookup_{} functions I may have missed). Reviewed By: zou3519 Differential Revision: D52067413
5c25e5a
to
1536491
Compare
This pull request was exported from Phabricator. Differential Revision: D52067413 |
This pull request has been merged in 51ee221. |
Summary: Title. Also the split_embedding_codegen_forward_[un]weighted_cuda ops as PT2 compliant (and mark split_embedding_codegen_lookup_{} functions I may have missed).
Differential Revision: D52067413