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

fp16 -> fp32 EmbeddingBag moved into CPU impl #47076

Closed
wants to merge 1 commit into from

Conversation

b-koopman
Copy link
Contributor

Summary: Eliminate double casting in glow when submitting fp16 per sample weights

Test Plan: buck test glow/glow/torch_glow/tests:embedding_bag_test

Reviewed By: allwu

Differential Revision: D24421367

Summary: Eliminate double casting in glow when submitting fp16 per sample weights

Test Plan: buck test glow/glow/torch_glow/tests:embedding_bag_test

Reviewed By: allwu

Differential Revision: D24421367

fbshipit-source-id: 69bf37925a3b0595cedd78a16b823198d3d57e82
@facebook-github-bot
Copy link
Contributor

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

@codecov
Copy link

codecov bot commented Oct 30, 2020

Codecov Report

Merging #47076 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #47076      +/-   ##
==========================================
- Coverage   68.85%   68.85%   -0.01%     
==========================================
  Files         436      436              
  Lines       56395    56395              
==========================================
- Hits        38833    38832       -1     
- Misses      17562    17563       +1     

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 7b8bd91.

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

2 participants