-
Notifications
You must be signed in to change notification settings - Fork 30
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
Add get_devices
method to dpctl.SyclPlatform
class
#1992
base: master
Are you sure you want to change the base?
Conversation
@oleksandr-pavlyk It occurred to me that |
View rendered docs @ https://intelpython.github.io/dpctl/pulls/1992/index.html |
Array API standard conformance tests for dpctl= ran successfully. |
d0d61c3
to
0629e08
Compare
Array API standard conformance tests for dpctl=0.19.0dev0=py310h93fe807_521 ran successfully. |
1 similar comment
Array API standard conformance tests for dpctl=0.19.0dev0=py310h93fe807_521 ran successfully. |
Array API standard conformance tests for dpctl=0.19.0dev0=py310h93fe807_523 ran successfully. |
@oleksandr-pavlyk Using these device types would yield the same result as
will always show 1. Same if To spare user confusion, I think it makes more sense to return empty lists for |
c5373d8
to
f60d32a
Compare
Also removes some remaining, sporadic references to the deprecated sycl::info::device_type::host
These device types act unexpectedly in DPC++ 2025.0.4, returning the same result as `sycl::info::device_type::all`, so use unknown device and return an empty list instead
f60d32a
to
1c8ee57
Compare
Array API standard conformance tests for dpctl=0.19.0dev0=py310h93fe807_541 ran successfully. |
This PR proposes implementing the
get_devices
method todpctl.SyclPlatform
class as per SYCL 2020 spec, which specifies including adevice_type
filter.Closes #509