-
Notifications
You must be signed in to change notification settings - Fork 22
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 missing stream
keyword argument to dpnp.ndarray.to_device
method
#2263
Conversation
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.
Everything is good, but we are not testing actual functionality of stream
keyword which purpose is to do synchronization.
We need to figure out how to properly test it.
Array API standard conformance tests for dpnp=0.17.0dev4=py312he4f9c94_12 ran successfully. |
@AlexanderKalistratov , I've applied the comments, please rereview. |
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.
LGTM!
Thank you @antonwolfy
…hod (#2263) The PR proposes to align `dpnp.ndarray.to_device` method with python array API and to add support of `stream` keyword argument. The tests coverage is extended to cover new argument. Previously muted array-api test is enabled back. Note, tests for `dpnp.__dlpack__` are updated due to recent changes done in [dpctl#1969](IntelPython/dpctl#1969). 6cc2348
The PR proposes to align
dpnp.ndarray.to_device
method with python array API and to add support ofstream
keyword argument.The tests coverage is extended to cover new argument. Previously muted array-api test is enabled back.
Note, tests for
dpnp.__dlpack__
are updated due to recent changes done in dpctl#1969.