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

Python bindings #8

Open
robashaw opened this issue Oct 1, 2020 · 3 comments · May be fixed by #51
Open

Python bindings #8

robashaw opened this issue Oct 1, 2020 · 3 comments · May be fixed by #51
Assignees
Milestone

Comments

@robashaw
Copy link
Owner

robashaw commented Oct 1, 2020

It would be useful to have libecpint available as an easily accessible python package. This would involve writing bindings to the main API.

@robashaw robashaw added this to the 1.1 milestone Oct 1, 2020
@robashaw robashaw self-assigned this Oct 1, 2020
@felipeZ
Copy link

felipeZ commented Feb 15, 2021

As part of the JOSS review, I would like to say that adding Python support is in my opinion one of the most important features to boost the impact of libecpint in the computational chemistry community.

Some thoughts about integrating libecpint with Python

  • I advice you to use Pybind11, because it is to use and it is a headers only library that is straightforward to integrate with cmake.
  • Maybe the PySCF community would be willing to add bindings to libecpint. Perhaps you would like to contact them.

@robashaw
Copy link
Owner Author

robashaw commented Mar 6, 2021

I am currently working on this using Pybind11 - thanks for the suggestion. Currently it appears to be working best (in terms of generating distributable packages) to have the python bindings as a separate repo pulling from this one. I will link the results here and in the documentation when it's ready.

@felipeZ
Copy link

felipeZ commented Mar 8, 2021

That is great!

@robashaw robashaw linked a pull request Sep 13, 2021 that will close this issue
@robashaw robashaw linked a pull request Sep 13, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants