-
Notifications
You must be signed in to change notification settings - Fork 4
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
Do we want to use Cython for wrapping? #3
Comments
👍 also consider cffi |
One argument for cython is that it has really nice built-in support for Numpy arrays--I think cffi is still lacking some of these features. My favorite part about the new Cython wrappers is that I can use array indexing to grab pointers. I know this sounds really dumb, but this is my favorite line of code ever:
|
What, you don't like computing pointers manually from stride arrays? |
That, @ihaque, is why when I do numerical C, I now use the gsl and functions like But in all seriousness, re-wrapping the code in cython should be a pretty low priority, given that the C wrapper already works just fine. As we move towards py3k (the c wrappers are slightly different format for py3k, but cython will handle that for you) or other python interpreters (pypy), then it becomes an issue again. But in the mean time, this is nbd. |
We could convert rmsdcalc.c to a cython wrapper, as in current MDTraj.
@rmcgibbo
The text was updated successfully, but these errors were encountered: