Skip to content

add error string in TORCH_CHECK #1000

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

Closed
wants to merge 6 commits into from

Conversation

jspark1105
Copy link
Contributor

Summary: Add more descriptive error strings to TORCH_CHECK in jagged ops

Reviewed By: jiaqizhai

Differential Revision: D34995563

Differential Revision: D34840552

fbshipit-source-id: c6837b141460d85f56c0185e5609823de2663270
Differential Revision: D34844686

fbshipit-source-id: 12b5f211be72f96a9f6181914ac5ad37a3ba1685
Differential Revision: D34845894

fbshipit-source-id: 5590c2c1f6e1d7ded5f453dc98bb5984d341659c
Differential Revision: D34876009

fbshipit-source-id: 4d845810f54ad804fcab1e25692637d66f6d1d75
Differential Revision: D34990893

fbshipit-source-id: 9c3bbddc25949d9ae66a325c838be8084757e83b
Summary: Add more descriptive error strings to TORCH_CHECK in jagged ops

Reviewed By: jiaqizhai

Differential Revision: D34995563

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

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

q10 added a commit to q10/FBGEMM that referenced this pull request Apr 10, 2025
Summary:
Pull Request resolved: facebookresearch/FBGEMM#1000

- Sometimes we would like to test FBGEMM OSS inside FB devservers.  However, the scripts currently assume an OSS CI environment with Ubuntu, and so the setup scripts fail with error, which this diff fixes.

X-link: pytorch#3909

Reviewed By: sryap

Differential Revision: D72255342

Pulled By: q10

fbshipit-source-id: c6361e8439269b553fdc55ebf01a04d283fea873
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.

2 participants