>>>> Sage passes doctests: Does that include detecting any leaks? The >>>> refnanny is not yet working, so I'd be wary of releasing until that is >>>> done unless we know that Sage doesn't leak more when Cython is upgraded. >>> I don't think Robert did any testing, but I can do some valgrinding with >>> the upcoming 3.3.alpha3 with the old and the new 0.11 snapshot to see if >>> anything pops up. It will be a day or two until I have results. >>> >> My refcount nanny now seems to be in a state where it doesn't crash >> things (though there are spurious reports still from not following the >> refnanny convention), and I've discovered at least one genuine leak so >> far in the cython test suite. > > Excuse my ignorance, but how do I turn on the refnanny in all the code? > I do not recall that being documented anywhere? I guess the best place > might be the wiki for that. Did anybody fix that leak you mentioned above?
No, I was just giving you a heads up for something I'm doing right now (and which is in experiment mode) -- running valgrind on Cython 0.11 may not be such a big point if I'm right about to get the refnanny in order, which will make those statistics outdated if the "refnanny effect" kicks in. I found the leak while playing with the refnanny: http://trac.cython.org/cython_trac/ticket/199 Establishing a baseline is always good though :-) Docs should be in wiki when it is more mature. There's some here: http://article.gmane.org/gmane.comp.python.cython.devel/4045/ Basically, once the Cython test suite runs with the nanny, this would be needed for SAGE: 1) Somebody should get distutils to compile Cython/Runtime/refnanny.pyx 2) SAGE needs to load refnanny.so or similar using RTLD_GLOBAL 3) Then, import it as a Python module 4) Then, run the C compiler with CYTHON_REFNANNY defined on Cython-generated sources Don't bother with it until it really works though :-) -- Dag Sverre _______________________________________________ Cython-dev mailing list [email protected] http://codespeak.net/mailman/listinfo/cython-dev
