Skip to content

fully revert tensor changes in dim_grid#460

Closed
JasonKChow wants to merge 1 commit into
facebookresearch:mainfrom
JasonKChow:export-D66374663
Closed

fully revert tensor changes in dim_grid#460
JasonKChow wants to merge 1 commit into
facebookresearch:mainfrom
JasonKChow:export-D66374663

Conversation

@JasonKChow
Copy link
Copy Markdown
Contributor

Summary: dim_grid tensor changes were only half reverted for some reason. This fully reverts it and fixes a bug where dim_grid was producing double the number of points when slicing.

Differential Revision: D66374663

@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 Nov 22, 2024
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

Summary:

dim_grid tensor changes were only half reverted for some reason. This fully reverts it and fixes a bug where dim_grid was producing double the number of points when slicing.

Differential Revision: D66374663
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request has been merged in aa36126.

JasonKChow added a commit that referenced this pull request Nov 25, 2024
Summary:
Pull Request resolved: #460

dim_grid tensor changes were only half reverted for some reason. This fully reverts it and fixes a bug where dim_grid was producing double the number of points when slicing.

Reviewed By: crasanders

Differential Revision: D66374663

fbshipit-source-id: c2226b59c6320bedf74c3374f6ae1347f47e4e26
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 Merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants