Skip to content

Use public for data owning class definitions #533

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
Merged
merged 1 commit into from
Aug 18, 2021
Merged

Use public for data owning class definitions #533

merged 1 commit into from
Aug 18, 2021

Conversation

oleksandr-pavlyk
Copy link
Contributor

Also introduce C-API functions to retrieve opaque DPCTL references
from Python objects.

This is useful in using dpctl entities as arguments in pybind11
applications

@oleksandr-pavlyk
Use public for data owning class definitions
7e14c55 
Also introduce C-API functions to retrieve opaque DPCTL references
from Python objects.

This is useful in using `dpctl` entities as arguments in pybind11
applications
@coveralls
Copy link
Collaborator

Coverage Status

Coverage decreased (-0.02%) to 64.882% when pulling 7e14c55 on feature/public-defs into db44872 on master.

@oleksandr-pavlyk oleksandr-pavlyk merged commit 9197366 into master Aug 18, 2021
@oleksandr-pavlyk oleksandr-pavlyk deleted the feature/public-defs branch August 20, 2021 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@diptorupd diptorupd Awaiting requested review from diptorupd

@reazulhoque reazulhoque Awaiting requested review from reazulhoque

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@oleksandr-pavlyk @coveralls