Skip to content
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

Migrate to pybind11 #393

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Migrate to pybind11 #393

wants to merge 1 commit into from

Conversation

inducer
Copy link
Owner

@inducer inducer commented Nov 6, 2022

Nothing works yet, very drafty.

@jakirkham
Copy link

It might be worth looking at nanobind, which is by the same author, but is a newer version of the approach

@inducer
Copy link
Owner Author

inducer commented Nov 21, 2022

I'm aware of it, having evaluated it for pyopencl (inducer/pyopencl#546). Migrating to pybind is a good stepping stone at any rate. For now, I appreciate that pybind is a more mature and more forgiving tool. And the rate demands of PyCUDA/PyOpenCL are not so great that the performance gains from nanobind materialize in real-world workloads, at least according to my measurements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants