-
Notifications
You must be signed in to change notification settings - Fork 24.4k
Update torch-xpu-ops commit pin #150409
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
Update torch-xpu-ops commit pin #150409
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/150409
Note: Links to docs will display an error until the docs builds have been completed. ❌ 5 New Failures, 1 Unrelated FailureAs of commit 401eb45 with merge base c65de03 ( NEW FAILURES - The following jobs have failed:
UNSTABLE - The following job is marked as unstable, possibly due to flakiness on trunk:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
To add the ciflow label This helps ensure we don't trigger CI on this PR until it is actually authorized to do so. Please ping one of the reviewers if you do not have access to approve and run workflows. |
@xytintel, could you please share the issue link of |
Adaptive_avg_pooling added an internal check and new UTs. Since torch-xpu-ops automatically enables these newly added UTs, it causes torch-xpu-ops to fail. We promptly captured this change and fixed it in torch-xpu-ops. (Relevant PR: #145241) |
The XPU linux CI test doesn't work caused by the issue #149995. We need land this commit pin update PR to fix XPU build issue and restore CI test after this PR land. |
@pytorchbot rebase -b main |
@pytorchbot started a rebase job onto refs/remotes/origin/main. Check the current status here |
Successfully rebased |
9925577
to
109fb6b
Compare
Move to #150554 |
Update the torch-xpu-ops commit to 11320f39484d1887870f24172c4803392491a76c, include
Bugfix of building error relating to XCCL. (Since [PGNCCL] Launch kernel on current stream & removerecord_stream
entirely #148590 has not landed, we decided to revert the XCCL changes to avoid breaking our CI.)