[Biopython-dev] PATCH: NumPy support for BioPython

Michiel de Hoon mdehoon at c2b2.columbia.edu
Sun Mar 11 19:32:32 UTC 2007


Ed Schofield wrote:
> It's a minor issue, but I can see several reasons to use the header file 
> NumPy provides for the purpose, rather than pasting its definitions into 
> our own source files:

I am not suggesting to paste the definitions in oldnumeric.h into our 
own source files. The point is that we don't need any of the definitions 
in oldnumeric.h. So we don't need to #include them, and we also don't 
need to paste them into Biopython.

For example, your patch adds these lines (among others) to 
Bio/KDTree/KDTree.i:

	#if (NDARRAY_VERSION >= 0x00090908)
	#include "numpy/oldnumeric.h"
	#endif

AFAICT, these three lines can simply be removed.

--Michiel.



More information about the Biopython-dev mailing list