-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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
[RFC][inductor][index_put] fallback to aten in torch deterministic mode #96898
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/96898
Note: Links to docs will display an error until the docs builds have been completed. ❗ 1 Active SEVsThere are 1 currently active SEVs. If your PR is affected, please view them below: ✅ No FailuresAs of commit 580c3a5: This comment was automatically generated by Dr. CI and updates every 15 minutes. |
working on failed tests |
0886373
to
0532033
Compare
wires crossed, #96695 is doing same thing pretty much, cc @colesbury |
oh, thanks for sharing! #96695 has dynamo support including guards. This PR only has inductor part for scatter_ops and index_put. I still see test failures and will look into them. |
@ngimel , it seems the index_put fallback will cause
|
Yeah, deterministic |
e58fb9a
to
246ce92
Compare
246ce92
to
580c3a5
Compare
drop the change to scatter/scatter_reduce fallback since it causes numerical issues, 1. we might need a |
@pytorchbot merge |
Merge failedReason: 1 mandatory check(s) are pending/not yet run. The first few are:
Dig deeper by viewing the pending checks on hud |
/easycla |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Fixes #93537
fallback to aten for index_put and scatter ops in torch deterministic mode
cc @soumith @voznesenskym @penguinwu @anijain2305 @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @Xia-Weiwen @wenzhe-nrv @jiayisunx @peterbell10 @desertfire