On Mon, Sep 28, 2009 at 3:51 AM, Dag Sverre Seljebotn <[email protected]> wrote: > 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 >
I appreciate that Dag. One day I'll understand all this stuff and I too will be able to help an OSS community... thats my goal. Again.. much thanks. ~Peter _______________________________________________ Cython-dev mailing list [email protected] http://codespeak.net/mailman/listinfo/cython-dev
