Changed the C wrapper in acalib.core.morph module

ChileanVirtualObservatory / acalib

Advanced Computing for Astronomy Library

http://acalib.readthedocs.io/en/latest/

GNU General Public License v3.0

7 stars 13 forks source link

Changed the C wrapper in acalib.core.morph module #38

Closed capgadsx closed 7 years ago

capgadsx commented 7 years ago

Removed the Cython wrapper
Added a new wrapper using the Python-C and Numpy-C API's only

Benchmark:

Done in Fedora 24 with Intel i5-2400 @ 3.10GHz
5 loops with 20 calls per loop

Results:

Wrapper	Min	Max	Avg
Cython	788ms	811ms	796ms
C API's	695ms	709ms	700ms

This means a 13% increase in performance and a more easy to read and understand wrapper code.

maray commented 7 years ago

As everybody that worked in pyCupid knows, I am big fan of C-Python (API) rather than Cython. I think this should be included ASAP, but let see what @Nesus says...