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

Add inverse_permute argument to permute_1D_sparse_data #1403

Closed
wants to merge 1 commit into from

Conversation

xing-liu
Copy link
Contributor

Summary: Variable batch size EC requires permute_1D backward. Add inverse_permute so we don't need to generate backward recat.

Reviewed By: YazhiGao

Differential Revision:
D40521813

LaMa Project: L1138451

@netlify
Copy link

netlify bot commented Oct 19, 2022

Deploy Preview for pytorch-fbgemm-docs canceled.

Name Link
🔨 Latest commit 3714170
🔍 Latest deploy log https://app.netlify.com/sites/pytorch-fbgemm-docs/deploys/6356da6e1c8b370009d1d785

@facebook-github-bot
Copy link
Contributor

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

xing-liu added a commit to xing-liu/FBGEMM that referenced this pull request Oct 24, 2022
Summary:
Pull Request resolved: pytorch#1403

Variable batch size EC requires permute_1D backward. Add inverse_permute so we don't need to generate backward recat.

Reviewed By: YazhiGao

Differential Revision:
D40521813

LaMa Project: L1138451

fbshipit-source-id: 92179093ac302bb3b794facfd3dde4491412058d
@facebook-github-bot
Copy link
Contributor

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

Summary:
Pull Request resolved: pytorch#1403

Variable batch size EC requires permute_1D backward. Add inverse_permute so we don't need to generate backward recat.

Reviewed By: YazhiGao

Differential Revision:
D40521813

LaMa Project: L1138451

fbshipit-source-id: 0f4a1b56ee92fafcec841bb3c9e14af26cced738
@facebook-github-bot
Copy link
Contributor

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

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