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

Pybind experiment #1728

Draft
wants to merge 20 commits into
base: next
Choose a base branch
from
Draft

Pybind experiment #1728

wants to merge 20 commits into from

Conversation

d7919
Copy link
Member

@d7919 d7919 commented May 22, 2019

For discussion.

Adds pybind11 as a submodule and provides an example of how to use this to use python within a C++ example. The commit history also includes a hacked example of using this as a part of the field generators so that we could use python as a part of the field function generation. This has been reverted in the final version as it was very much a hack.

In the long run we may want to put these into a separate variable as
only required when building the wrappers
Require enable-shared=yes if we turn on pybind11
Adds a helper struct for compile time compatibility checking.
This reverts commit 98d9d78.

This commit serves as a record of one way to approach this. In the long
run a larger rewrite is probably more useful and PR #1494 already
provides some changes in this area.
@dschwoerer dschwoerer added proposal A code/feature outline proposal merge-conflicts PR with significant merge conflicts that need to be resolved labels Oct 16, 2023
@dschwoerer
Copy link
Contributor

Needs to be adopted for cmake.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-conflicts PR with significant merge conflicts that need to be resolved proposal A code/feature outline proposal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants