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

Enable jit script for SSD TBE (for GPU inference e2e integration) #1533

Closed
wants to merge 1 commit into from

Conversation

jianyuh
Copy link
Member

@jianyuh jianyuh commented Jan 1, 2023

Summary:
Address several issues on JIT Scripting SSD TBE op (preparing for GPU inference e2e integration).

  • Inline to_pinned_cpu
  • Refactor split_embedding_weights to make it jit scriptable.
  • Add/fix more type annotation (required for JIT).
  • Some C++ type conversion convention fix.

Differential Revision: D42300244

@netlify
Copy link

netlify bot commented Jan 1, 2023

Deploy Preview for pytorch-fbgemm-docs canceled.

Name Link
🔨 Latest commit 1e154b1
🔍 Latest deploy log https://app.netlify.com/sites/pytorch-fbgemm-docs/deploys/63b0d7c4eb60e00008d66e81

@facebook-github-bot
Copy link
Contributor

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

…torch#1533)

Summary:
Pull Request resolved: pytorch#1533

Address several issues on JIT Scripting SSD TBE op (preparing for GPU inference e2e integration).

- Inline to_pinned_cpu
- Refactor split_embedding_weights to make it jit scriptable.
- Add/fix more type annotation (required for JIT).
- Some C++ type conversion convention fix.

Differential Revision: D42300244

fbshipit-source-id: 26226372ef0305be348ef3108edcd9a3740769cb
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in e0b80d1.

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.

None yet

3 participants