> Robert wrote:
>> Robert wrote:
>>>
>>> this and more related cleanup, speedup of importer, --inplace
>>> --verbose reload() .. here now:
>>>
>>> http://trac.cython.org/cython_trac/ticket/312
>>>
>>
>> maybe sb likes to test it further:
>>
>> * support for dynamic "reload(<pyxmodule>)" added: e.g change of
>> the Cython code without stopping & relaunching a big app.
>> <so_path>.reloadNN files may arise, as previously loaded files are
>> blocked.
>
> This is great; I've always wanted reload to work...
>
> Reading over the current patch, I see some commented-out code which should
> be removed etc.
>
> How ready to you consider this yourself? Should I include it in Cython
> 0.11.2? (I'm certainly tempted to.)

Hmm. I just had a segfault with reload_support:

In [1]: import pyximport

In [2]: pyximport.install(build_dir='.', reload_support=True)

In [3]: import integrate_sinsq

In [4]: integrate_sinsq.integrate_sinsq(0, 1, 100)
hello
Out[4]: 0.30606995203805437

In [5]: reload(integrate_sinsq)
Out[5]: <module 'integrate_sinsq' from
'./lib.linux-x86_64-2.5/integrate_sinsq.so.reload0'>

In [6]: integrate_sinsq.integrate_sinsq(0, 1, 100)
Segmentation fault

The contents of integrate_sinsq is trivial, although it does have a call
so "sin" in math.h (I didn't link m, I think Python loads this for the
module -- at least it works before the reload).

Dag Sverre

_______________________________________________
Cython-dev mailing list
Cython-dev@codespeak.net
http://codespeak.net/mailman/listinfo/cython-dev

Reply via email to