-
Notifications
You must be signed in to change notification settings - Fork 24.2k
xpu: torch.xpu.get_arch_list() to return [] if xpu not compiled #147431
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
Initially discussed here: pytorch#132945 (comment) Previously `torch.xpu.get_arch_list()` got relaxed to work even if XPU device is not available. However, we overlooked the case when pytorch is not compiled with XPU support. In such a case function throws an exception. This commit adjusts this behavior and makes function return `[]` even if pytorch is not compiled with XPU support. Signed-off-by: Dmitry Rogozhkin <dmitry.v.rogozhkin@intel.com>
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/147431
Note: Links to docs will display an error until the docs builds have been completed. ❌ 3 New FailuresAs of commit c290b5b with merge base 525ca80 ( NEW FAILURES - The following jobs have failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
@pytorchbot label "topic: not user facing" |
"The failures are unrelated to this PR." |
Merge startedYour change will be merged while ignoring the following 3 checks: xpu / linux-jammy-xpu-2025.0-py3.9 / test (default, 4, 4, linux.idc.xpu), trunk / linux-focal-rocm6.3-py3.10 / test (default, 1, 2, linux.rocm.gpu.2), trunk / linux-focal-rocm6.3-py3.10 / test (default, 2, 2, linux.rocm.gpu.2) Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
) Initially discussed here: #132945 (comment) Previously `torch.xpu.get_arch_list()` got relaxed to work even if XPU device is not available. However, we overlooked the case when pytorch is not compiled with XPU support. In such a case function throws an exception. This commit adjusts this behavior and makes function return `[]` even if pytorch is not compiled with XPU support. CC: @EikanWang @fengyuan14 @guangyey @malfet @albanD Pull Request resolved: #147431 Approved by: https://github.com/guangyey, https://github.com/EikanWang, https://github.com/albanD
…rch#147431) Initially discussed here: pytorch#132945 (comment) Previously `torch.xpu.get_arch_list()` got relaxed to work even if XPU device is not available. However, we overlooked the case when pytorch is not compiled with XPU support. In such a case function throws an exception. This commit adjusts this behavior and makes function return `[]` even if pytorch is not compiled with XPU support. CC: @EikanWang @fengyuan14 @guangyey @malfet @albanD Pull Request resolved: pytorch#147431 Approved by: https://github.com/guangyey, https://github.com/EikanWang, https://github.com/albanD
Initially discussed here: #132945 (comment)
Previously
torch.xpu.get_arch_list()
got relaxed to work even if XPU device is not available. However, we overlooked the case when pytorch is not compiled with XPU support. In such a case function throws an exception. This commit adjusts this behavior and makes function return[]
even if pytorch is not compiled with XPU support.CC: @EikanWang @fengyuan14 @guangyey @malfet @albanD