-
Notifications
You must be signed in to change notification settings - Fork 26.3k
[BE] Update CUDNN to 9.4.0.58 #127925
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
[BE] Update CUDNN to 9.4.0.58 #127925
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/127925
Note: Links to docs will display an error until the docs builds have been completed. ❌ 13 New Failures, 1 Cancelled Job, 2 Unrelated FailuresAs of commit 6a28122 with merge base f8a5b71 ( NEW FAILURES - The following jobs have failed:
CANCELLED JOB - The following job was cancelled. Please retry:
FLAKY - The following job failed but was likely due to flakiness present on trunk:
BROKEN TRUNK - The following job failed but was present on the merge base:👉 Rebase onto the `viable/strict` branch to avoid these failures
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
In the off chance we can squeeze this into to 2.4 or if it solves any inductor issues we are having. |
|
@pytorchbot rebase |
|
@pytorchbot started a rebase job onto refs/remotes/origin/viable/strict. Check the current status here |
|
Successfully rebased |
0557788 to
159c0c8
Compare
159c0c8 to
decaa27
Compare
decaa27 to
a5f9cfe
Compare
a5f9cfe to
0d6ad43
Compare
|
@atalman Updated it again to the latest CUDNN version |
0d6ad43 to
299ca52
Compare
|
We have a few internal (conv?) tests that seem to newly failiying with 9.3.0.75, will try to provide public reproducers that we can test... |
|
@eqy Are these issues fixed in the newly released 9.4? |
75d27aa to
c1d95f1
Compare
c2c03a1 to
2e46d77
Compare
|
Tentatively signing off on this, I think we are OK for now |
|
Have inductor tests been run with this PR yet? It would be good to know whether certain models would have fluctuations regarding the accuracy. |
892e328 to
ef99783
Compare
|
@nWEIdia Updated benchmarks, tinynet_a now passes inductor accuracy tests |
nWEIdia
left a comment
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 great to me!
ef99783 to
6a28122
Compare
|
@nWEIdia I need to get the binaries uploaded to the pytorch_builder repo, that's the main blocker. |
cc @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @chenyang78 @kadeng @chauhang @amjames @rec