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 variable batch per feature support to EBC (tw/cw only) #1358

Closed
wants to merge 1 commit into from

Conversation

joshuadeng
Copy link
Contributor

Summary:
To enable provide stride_per_key_per_rank and reverse_indices to the KJT

Reindexing is done internally with EBC and with a user provided reverse indices tensor

Differential Revision: D48805440

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 30, 2023
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

joshuadeng added a commit to joshuadeng/torchrec that referenced this pull request Aug 30, 2023
)

Summary:
Pull Request resolved: pytorch#1358

X-link: pytorch/FBGEMM#1986

To enable provide stride_per_key_per_rank and reverse_indices to the KJT

Reindexing is done internally with EBC and with a user provided reverse indices tensor

Differential Revision: D48805440

fbshipit-source-id: 3f9439b0ca5325489fcf9c629bffe95da53ece9c
joshuadeng added a commit to joshuadeng/FBGEMM that referenced this pull request Aug 30, 2023
)

Summary:
X-link: pytorch/torchrec#1358

Pull Request resolved: pytorch#1986

To enable provide stride_per_key_per_rank and reverse_indices to the KJT

Reindexing is done internally with EBC and with a user provided reverse indices tensor

Differential Revision: D48805440

fbshipit-source-id: 504f1dd0ebe0c2de28fe4d2bc623e3ba3b4da69a
@facebook-github-bot
Copy link
Contributor

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

joshuadeng added a commit to joshuadeng/torchrec that referenced this pull request Aug 30, 2023
)

Summary:
Pull Request resolved: pytorch#1358

X-link: pytorch/FBGEMM#1986

To enable provide stride_per_key_per_rank and reverse_indices to the KJT

Reindexing is done internally with EBC and with a user provided reverse indices tensor

Differential Revision: D48805440

fbshipit-source-id: 2cb3665209025a44e659e5277534ad3f9e41ae0d
joshuadeng added a commit to joshuadeng/FBGEMM that referenced this pull request Aug 30, 2023
)

Summary:
X-link: pytorch/torchrec#1358

Pull Request resolved: pytorch#1986

To enable provide stride_per_key_per_rank and reverse_indices to the KJT

Reindexing is done internally with EBC and with a user provided reverse indices tensor

Differential Revision: D48805440

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

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

joshuadeng added a commit to joshuadeng/torchrec that referenced this pull request Aug 30, 2023
)

Summary:
Pull Request resolved: pytorch#1358

X-link: pytorch/FBGEMM#1986

To enable provide stride_per_key_per_rank and reverse_indices to the KJT

Reindexing is done internally with EBC and with a user provided reverse indices tensor

Differential Revision: D48805440

fbshipit-source-id: b151328e3d2e8ec6cae4f235db505d9a3a70e3fd
joshuadeng added a commit to joshuadeng/FBGEMM that referenced this pull request Aug 31, 2023
)

Summary:
X-link: pytorch/torchrec#1358

Pull Request resolved: pytorch#1986

To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with a user provided inverse indices tensor

we provide a utility function `merge_inverse_indices_per_key` to convert inverse indices in format `Dict[str, torch.Tensor]` to `torch.Tensor` accepted by KJT constructor

Differential Revision: D48805440

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

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

joshuadeng added a commit to joshuadeng/torchrec that referenced this pull request Aug 31, 2023
)

Summary:
Pull Request resolved: pytorch#1358

X-link: pytorch/FBGEMM#1986

To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with a user provided inverse indices tensor

we provide a utility function `merge_inverse_indices_per_key` to convert inverse indices in format `Dict[str, torch.Tensor]` to `torch.Tensor` accepted by KJT constructor

Differential Revision: D48805440

fbshipit-source-id: f66d4ec17457780922d8d63a8b71d45bd272c535
joshuadeng added a commit to joshuadeng/torchrec that referenced this pull request Sep 2, 2023
)

Summary:

X-link: pytorch/FBGEMM#1986

To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with a user provided inverse indices tensor

we provide a utility function `merge_inverse_indices_per_key` to convert inverse indices in format `Dict[str, torch.Tensor]` to `torch.Tensor` accepted by KJT constructor

Differential Revision: D48805440
@facebook-github-bot
Copy link
Contributor

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

joshuadeng added a commit to joshuadeng/torchrec that referenced this pull request Sep 2, 2023
)

Summary:

X-link: pytorch/FBGEMM#1986

To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with a user provided inverse indices tensor

we provide a utility function `merge_inverse_indices_per_key` to convert inverse indices in format `Dict[str, torch.Tensor]` to `torch.Tensor` accepted by KJT constructor

Differential Revision: D48805440
@facebook-github-bot
Copy link
Contributor

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

joshuadeng added a commit to joshuadeng/torchrec that referenced this pull request Oct 9, 2023
Summary:

X-link: pytorch/FBGEMM#1986

To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with user provided inverse indices

Differential Revision: D48805440
joshuadeng added a commit to joshuadeng/torchrec that referenced this pull request Oct 9, 2023
Summary:

X-link: pytorch/FBGEMM#1986

To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with user provided inverse indices

Differential Revision: D48805440
@facebook-github-bot
Copy link
Contributor

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

1 similar comment
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

joshuadeng added a commit to joshuadeng/FBGEMM that referenced this pull request Nov 20, 2023
Summary:
X-link: pytorch/torchrec#1358


To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with user provided inverse indices

Reviewed By: bigning

Differential Revision: D48805440
joshuadeng added a commit to joshuadeng/torchrec that referenced this pull request Nov 20, 2023
Summary:

X-link: pytorch/FBGEMM#1986

To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with user provided inverse indices

Reviewed By: bigning

Differential Revision: D48805440
@facebook-github-bot
Copy link
Contributor

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

joshuadeng added a commit to joshuadeng/FBGEMM that referenced this pull request Nov 20, 2023
Summary:
X-link: pytorch/torchrec#1358


To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with user provided inverse indices

Reviewed By: bigning

Differential Revision: D48805440
joshuadeng added a commit to joshuadeng/torchrec that referenced this pull request Nov 20, 2023
Summary:

X-link: pytorch/FBGEMM#1986

To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with user provided inverse indices

Reviewed By: bigning

Differential Revision: D48805440
@facebook-github-bot
Copy link
Contributor

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

1 similar comment
@facebook-github-bot
Copy link
Contributor

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

joshuadeng added a commit to joshuadeng/torchrec that referenced this pull request Nov 20, 2023
Summary:

X-link: pytorch/FBGEMM#1986

To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with user provided inverse indices

Reviewed By: bigning

Differential Revision: D48805440
joshuadeng added a commit to joshuadeng/torchrec that referenced this pull request Nov 20, 2023
Summary:

X-link: pytorch/FBGEMM#1986

To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with user provided inverse indices

Reviewed By: bigning

Differential Revision: D48805440
joshuadeng added a commit to joshuadeng/FBGEMM that referenced this pull request Nov 20, 2023
Summary:
X-link: pytorch/torchrec#1358


To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with user provided inverse indices

Reviewed By: bigning

Differential Revision: D48805440
joshuadeng added a commit to joshuadeng/torchrec that referenced this pull request Nov 20, 2023
Summary:

X-link: pytorch/FBGEMM#1986

To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with user provided inverse indices

Reviewed By: bigning

Differential Revision: D48805440
@facebook-github-bot
Copy link
Contributor

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

joshuadeng added a commit to joshuadeng/FBGEMM that referenced this pull request Nov 20, 2023
Summary:
X-link: pytorch/torchrec#1358


To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with user provided inverse indices

Reviewed By: bigning

Differential Revision: D48805440
joshuadeng added a commit to joshuadeng/torchrec that referenced this pull request Nov 20, 2023
Summary:

X-link: pytorch/FBGEMM#1986

To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with user provided inverse indices

Reviewed By: bigning

Differential Revision: D48805440
@facebook-github-bot
Copy link
Contributor

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

1 similar comment
@facebook-github-bot
Copy link
Contributor

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

joshuadeng added a commit to joshuadeng/FBGEMM that referenced this pull request Nov 20, 2023
Summary:
X-link: pytorch/torchrec#1358


To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with user provided inverse indices

Reviewed By: bigning

Differential Revision: D48805440
Summary:

X-link: pytorch/FBGEMM#1986

To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with user provided inverse indices

Reviewed By: bigning

Differential Revision: D48805440
@facebook-github-bot
Copy link
Contributor

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

joshuadeng added a commit to joshuadeng/FBGEMM that referenced this pull request Nov 20, 2023
Summary:
X-link: pytorch/torchrec#1358

Pull Request resolved: pytorch#1986

To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with user provided inverse indices

Reviewed By: bigning

Differential Revision: D48805440

fbshipit-source-id: 9f49f7789ea2a20344cc56f84018fc761892836d
joshuadeng added a commit to joshuadeng/FBGEMM that referenced this pull request Nov 20, 2023
Summary:
X-link: pytorch/torchrec#1358


To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with user provided inverse indices

Reviewed By: bigning

Differential Revision: D48805440
facebook-github-bot pushed a commit to pytorch/FBGEMM that referenced this pull request Nov 20, 2023
Summary:
X-link: pytorch/torchrec#1358

Pull Request resolved: #1986

To enable provide stride_per_key_per_rank and inverse_indices to the KJT

Reindexing is done internally with EBC and with user provided inverse indices

Reviewed By: bigning

Differential Revision: D48805440

fbshipit-source-id: e490baa41b9c91b02e6c246940d4a3a2c4551d65
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants