Skip to content

Implement a separate kernel for array-based index remapping lookup. #682

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

Closed
wants to merge 1 commit into from

Conversation

ipiszy
Copy link
Contributor

@ipiszy ipiszy commented Sep 1, 2021

Summary:
This diff:

  • implements an array-based lookup kernel for embedding index remapping and makes it default;
  • adds a new benchmark for array-based pruning lookup;
  • adds new features into the nbit-device benchmark to run multiple iterations and ignore the warmup runs;

Differential Revision: D30556183

@facebook-github-bot
Copy link
Contributor

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

…ytorch#682)

Summary:
Pull Request resolved: pytorch#682

This diff:
* implements an array-based lookup kernel for embedding index remapping and makes it default;
* adds a new benchmark for array-based pruning lookup;
* adds new features into the nbit-device benchmark to run multiple iterations and ignore the warmup runs;

Reviewed By: ajtulloch

Differential Revision: D30556183

fbshipit-source-id: 58be36cab7076a2a62905c4181b6234a1dbf47b7
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 820b4c7.

q10 pushed a commit to q10/FBGEMM that referenced this pull request Apr 10, 2025
Summary:
X-link: pytorch#3601

Pull Request resolved: facebookresearch/FBGEMM#682

Fix type to assert group size

Reviewed By: jianyuh

Differential Revision: D68499555

fbshipit-source-id: 58ac6869189924610d03ae7339ca3d24b1898fed
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