Add support for int64_t indices and offsets in TBE inference [11/N]#3264

Closed

q10 wants to merge 1 commit into

pytorch:mainfrom

q10:export-D64725653

q10 commented Oct 22, 2024

Contributor

Summary: This is a re-land of parts of D63778645 (7C) combined with D63807049 (8). They were reverted earlier due to S460368 (D64618134 and D64618221), but with D64619774 (9) and D64705072(19) in place, the issues with deploying int64_t-based index remap arrays should be resolved, at least on the FBGEMM side

Differential Revision: D64725653

facebook-github-bot added the cla signed label

facebook-github-bot commented Oct 22, 2024

Contributor

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

facebook-github-bot added the fb-exported label

netlify Bot commented Oct 22, 2024 •

edited

Loading

❌ Deploy Preview for pytorch-fbgemm-docs failed.

Name	Link
🔨 Latest commit	`5021c9c`
🔍 Latest deploy log	https://app.netlify.com/sites/pytorch-fbgemm-docs/deploys/671bed92bf9e9f00083c42a3

facebook-github-bot commented Oct 22, 2024

Contributor

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

9 similar comments

facebook-github-bot commented Oct 24, 2024

Contributor

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

facebook-github-bot commented Oct 24, 2024

Contributor

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

facebook-github-bot commented Oct 24, 2024

Contributor

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

facebook-github-bot commented Oct 25, 2024

Contributor

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

facebook-github-bot commented Oct 25, 2024

Contributor

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

facebook-github-bot commented Oct 25, 2024

Contributor

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

facebook-github-bot commented Oct 25, 2024

Contributor

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

facebook-github-bot commented Oct 25, 2024

Contributor

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

facebook-github-bot commented Oct 25, 2024

Contributor

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


          Add support for int64_t indices and offsets in TBE inference [12/N] (#…

5c65cb1

…3264)

Summary:

X-link: facebookresearch/FBGEMM#365

Parent diff is D64777535

This is a re-land of parts of D63778645 (7C) combined with D63807049 (8).  They were reverted earlier due to S460368 (D64618134 and D64618221), but with D64619774 (9) D64705072 (10), and D64777535 (11) in place, the issues with deploying `int64_t`-based index remap arrays should be resolved, at least on the FBGEMM side

Differential Revision: D64725653

facebook-github-bot commented Oct 26, 2024

Contributor

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

facebook-github-bot closed this in

00b55af

facebook-github-bot added the Merged label

facebook-github-bot commented Oct 29, 2024

Contributor

This pull request has been merged in 00b55af.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla signed fb-exported Merged