On Thu, Jun 19, 2008 at 5:06 PM, Robert Kern <[EMAIL PROTECTED]> wrote:
> On Thu, Jun 19, 2008 at 19:02, Stéfan van der Walt <[EMAIL PROTECTED]> wrote:
>> 2008/6/19 Fernando Perez <[EMAIL PROTECTED]>:
>>>> - import_array().  Can we put it at the bottom of c_numpy.pxd, so that
>>>> users don't have to import it every time?
>>>
>>> Sounds fine to me, but I don't know if there could be a subtle problem
>>> with pxds used by multiple pyx files.  I'm not comfortable enough with
>>> cython to answer the question.
>>
>> >From what I understand, a .pxd file is loaded only once -- is that
>> correct?  Either way, I don't think executing import_array more than
>> once could hurt (or could it -- Robert?).
>
> I don't think it would cause bad things to happen. It is slightly
> suboptimal, but only just.

As long as it doesn't cause problems, then I'd vote for doing it.
This only happens at module import time, so even if several modules
using cython call it, it's only when they get imported.  I think the
API simplification is worth it (minor, but still nice).

I just tested the changes and there seem to be no ill effects.  Should
I  go ahead and commit it? I'll do it in a single commit with no other
changes so it's easy to revert should it prove to be a bad idea.

Cheers,

f
_______________________________________________
Numpy-discussion mailing list
Numpy-discussion@scipy.org
http://projects.scipy.org/mailman/listinfo/numpy-discussion

Reply via email to