Dag Sverre Seljebotn wrote: > Peter Alexander wrote: >> On Mon, Sep 28, 2009 at 2:40 AM, Dag Sverre Seljebotn >> <[email protected]> wrote: >>> Peter Alexander wrote: >>>> Hi all. >>>> >>>> There is a new 'metadata' field in the PyArray_Descr struct now >>>> prevalent in numpy trunk... >>>> >>>> This has been originated for the new 'datetime' array type and is to >>>> be useful elsewhere. >>> OK... is there something you thing Cython should do about it? Patches >>> welcome. >>> >>> I don't think many Cython/NumPy users touch the "dtype" type in >>> numpy.pxd, and accessing the metadata through the Python layer can often >>> be as fast, depending on what type it is (and if it is available through >>> the Python layer). >>> >>> There's plenty of things in NumPy which is not in numpy.pxd BTW, just >>> because I figured I'd spend my times on other things in Cython until >>> somebody actually complain about missing functionality. >>> >> I didn't mean to upset you Dag, it certainly wasn't my intention. My >> sincere apologies. >> >> I just found out that my cython modules could not be imported because >> of the new dtype structure and I had to resort back to numpy-1.3.0... >> no big deal.. >> >> Anyway I thought that I would simply let you know about it, is all. I >> am self-taught in computer science, and if I was smart enough to >> provide a patch, I certainly would have. I totally respect what you >> all do... please don't get me wrong. > > I'm terribly sorry, I was in a hurry and it wasn't the intention at all > to communicate that I didn't value your feedback (which I do). > > I simply didn't understand what you wanted and thought it would be > easier to explain with a patch... > > If you cannot import Cython modules with the trunk NumPy, that must > indeed be fixed, preferably for the 0.12 release. Thanks for the heads > up. This is now > > http://trac.cython.org/cython_trac/ticket/378 > > I'll have a look when I get time. > > (As for the patch: If you are able to modify Cython/Include/numpy.pxd so > that it works with trunk NumPy, simply type "hg diff > Cython/Include/numpy.pxd > myfile" and attach myfile to an email to this > list -- provided you have Mercurial installed on your computer. But as I > said, since NumPy doesn't run it's high priority and I'll have a look > myself eventually.). >
Heh, just realised that what you said might have been that you wouldn't know what to change in numpy.pxd, not that you didn't know how to make the patch file :-) Sorry :-) Anyway, thanks again for the report. -- Dag Sverre _______________________________________________ Cython-dev mailing list [email protected] http://codespeak.net/mailman/listinfo/cython-dev
