-
Notifications
You must be signed in to change notification settings - Fork 30
Update for cython 3 #1302
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 for cython 3 #1302
Conversation
Changes per https://cython.readthedocs.io/en/latest/src/userguide/external_C_code.html Capsule deleter functions are marked noexcept except/noexcept must be specified before gil/no gil. cpdef functions should not have except qualifiers.
Added ``` ``` to quieten mightly flood of warnings.
This reverts commit 03a46e1.
View rendered docs @ https://intelpython.github.io/dpctl/pulls/1302/index.html |
Array API standard conformance tests for dpctl=0.14.6dev0=py310h7bf5fec_31 ran successfully. |
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.
Checked with dpnp build, it works fine, no dpctl relating warning. Thank you @oleksandr-pavlyk !
I should also add that after this PR the source remains buildable with Cython 0.29.36 |
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 good to me!
With this we can look ahead to enabling Python arithmetic operators for usm_ndarray
s.
Deleted rendered PR docs from intelpython.github.com/dpctl, latest should be updated shortly. 🤞 |
Array API standard conformance tests for dpctl=0.14.6dev0=py310h7bf5fec_65 ran successfully. |
This PR closes gh-1277, reverts 03a46e1 to enable Cython 3.0, modifies Cython files per Cython guidance, and removes deprecated in 2023.2
sycl::aspect::usm_restricted_shared_allocations
anddpctl.SyclDevice.has_aspect_usm_restricted_shared_allocations
property.