-
Notifications
You must be signed in to change notification settings - Fork 25.7k
[CUDA] Workaround shmem limit for certain input sizes in AdaptiveAvgPool1D
#115231
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
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/115231
Note: Links to docs will display an error until the docs builds have been completed. ✅ You can merge normally! (3 Unrelated Failures)As of commit 23323f3 with merge base 87ea6fb ( FLAKY - The following jobs failed but were likely due to flakiness present on trunk:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
Also related: #70701 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks ok to me and failures look unrelated let's maybe hold off on merging till @marcmk6 verifies that it fixes the issue
@pytorchmergebot rebase |
@pytorchbot started a rebase job onto refs/remotes/origin/viable/strict. Check the current status here |
Successfully rebased |
7e51d02
to
e7586f5
Compare
@eqy @mikaylagawarecki |
@marcmk6 I was able to reproduce the error and the test (uses the same repro) passes. However we could check that it suffices for your particular hardware: what model(s) of GPU are you using? |
@eqy I got RTX A5000 and an old Tesla P4 |
@pytorchmergebot rebase |
1 similar comment
@pytorchmergebot rebase |
@pytorchbot started a rebase job onto refs/remotes/origin/viable/strict. Check the current status here |
Successfully rebased |
e7586f5
to
23323f3
Compare
I've verified that this is fixed on P100 and A40 (the closest GPUs I have access to compared to P4 and A5000), merging. @pytorchmergebot merge |
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 |
…Pool1D` (pytorch#115231) Reference issue pytorch#68248 CC @ptrblck @malfet @xwang233 Pull Request resolved: pytorch#115231 Approved by: https://github.com/mikaylagawarecki
Reference issue #68248
CC @ptrblck @malfet @xwang233
cc @albanD @mruberry @jbschlosser @walterddr @mikaylagawarecki @ptrblck