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

Turn on opcheck for split_table_batched_embeddings_test #2092

Closed
wants to merge 4 commits into from

Conversation

ezyang
Copy link
Contributor

@ezyang ezyang commented Oct 24, 2023

Differential Revision: D50613728

@netlify
Copy link

netlify bot commented Oct 24, 2023

Deploy Preview for pytorch-fbgemm-docs canceled.

Name Link
🔨 Latest commit d2859ae
🔍 Latest deploy log https://app.netlify.com/sites/pytorch-fbgemm-docs/deploys/65396d7ae620d700088668f6

@facebook-github-bot
Copy link
Contributor

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

ezyang added a commit to ezyang/FBGEMM that referenced this pull request Oct 24, 2023
Summary: Pull Request resolved: pytorch#2092

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

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

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

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

ezyang added a commit to ezyang/FBGEMM that referenced this pull request Oct 25, 2023
Summary: Pull Request resolved: pytorch#2092

Differential Revision: D50613728

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

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

ezyang added a commit to ezyang/FBGEMM that referenced this pull request Oct 25, 2023
Summary: Pull Request resolved: pytorch#2092

Differential Revision: D50613728

fbshipit-source-id: 40b2d898fc0c83abc10e6a74f5e73da9fed331c0
ezyang added a commit to ezyang/FBGEMM that referenced this pull request Oct 25, 2023
Summary: Pull Request resolved: pytorch#2092

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

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

ezyang added a commit to ezyang/FBGEMM that referenced this pull request Oct 25, 2023
Summary: Pull Request resolved: pytorch#2092

Differential Revision: D50613728
ezyang added a commit to ezyang/FBGEMM that referenced this pull request Oct 25, 2023
Summary: Pull Request resolved: pytorch#2092

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

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

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

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

ezyang added a commit to ezyang/FBGEMM that referenced this pull request Oct 25, 2023
Summary: Pull Request resolved: pytorch#2092

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

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

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

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

Summary: Pull Request resolved: pytorch#2087

Reviewed By: zou3519

Differential Revision: D50584541
Summary: Pull Request resolved: pytorch#2090

Reviewed By: zou3519

Differential Revision: D50586828
Summary:

I'm not entirely sure about some of these (e.g., bin_ctr_in_use_after) but it should
be harmless enough to accept symbolic SymInt at all these sites.

Reviewed By: zou3519

Differential Revision: D50587015
Summary: Pull Request resolved: pytorch#2092

Reviewed By: zou3519

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

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 6c2be88.

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