Skip to content

get_current_queue and other functions should be exposed as cpdef functions #112

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.

Sign up for GitHub

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

Jump to bottom
Closed
diptorupd opened this issue Oct 6, 2020 · 1 comment · Fixed by #150
Closed

get_current_queue and other functions should be exposed as cpdef functions #112

diptorupd opened this issue Oct 6, 2020 · 1 comment · Fixed by #150
Assignees
@diptorupd @1e-to
Milestone
gold

Comments

@diptorupd
Copy link
Contributor

dpctl functions get_current_queue, get_current_device, get_current_backend are not exposed as C functions. Other Cython modules that want to use dpctl cannot do a cimport to access these functions.

We should redefine these functions as cpdef and expose them via _sycl_core.pxd.
@diptorupd diptorupd added this to the gold milestone Oct 6, 2020
@oleksandr-pavlyk
Copy link
Contributor

#125 exports get_current_queue.

@1e-to 1e-to mentioned this issue Oct 21, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@diptorupd diptorupd

@1e-to 1e-to

Labels
None yet
Projects
None yet
Milestone
gold
Development

Successfully merging a pull request may close this issue.

Change funcs to cpdef 1e-to/dpctl
3 participants
@diptorupd @1e-to @oleksandr-pavlyk