On Oct 25, 8:04 pm, Robert Bradshaw <[EMAIL PROTECTED]>
wrote:
> On Oct 25, 2007, at 10:03 AM, mabshoff wrote:
>
>
>
> > On Oct 25, 11:25 am, mabshoff <[EMAIL PROTECTED]
> > dortmund.de> wrote:
> >> On Oct 25, 9:06 am, Robert Bradshaw <[EMAIL PROTECTED]>
> >> wrote:
>
> >> Hi Robert,
>
> >>> Seehttp://sage.math.washington.edu/home/robertwb/cython/Alltests
> >>> pass with cleanup level 1 (now default). The command line option
>
> >>> --cleanup n
>
> >>> sets the level (0 <= n <= 3). Things crash on quit (in SAGE, though
> >>> not with simple examples) with level 3. This may or may not fix
> >>> #557,
> >>> but is a start (but at the very least should make the memcheck logs
> >>> somewhat cleaner). We can play with it from here).
>
> >> Excellent, I will play with this tonight and report back.
>
> > Hmm, with 2.8.9 I get the following error when using
>
> >http://sage.math.washington.edu/home/robertwb/cython/
> > cython-0.9.6.8.spkg
>
> > Building sage/matrix/misc.c because it depends on sage/matrix/
> > misc.pyx.
> > touch sage/matrix/misc.pyx; cython --embed-positions --incref-local-
> > binop -I/tmp/Work-mabshoff/sage-2.8.9/devel/sage-main -o sage/matrix/
> > misc.c sage/matrix/misc.pyx
>
> > Error converting Pyrex file to C:
> > ------------------------------------------------------------
> > ...
> >                                      0, 0, 0)
> >     cdef mpz_t* L_row
> >     cdef mod_int* A_row
> >     for i from 0 <= i < A._nrows:
> >         L_row = L._matrix[i]
> >         A_row = A._matrix[i]
> >                         ^
> > ------------------------------------------------------------
>
> > /tmp/Work-mabshoff/sage-2.8.9/devel/sage-main/sage/matrix/misc.pyx:
> > 54:25: Cannot assign type 'sage.matrix.matrix_modn_dense.mod_int *' to
> > 'sage.matrix.misc.mod_int *'
> > sage: Error running cython.
> > sage: There was an error installing modified sage library code.
>
> > I don't think misc.pyx has changed in a while. Any ideas?
>

Hi Robert,

> Did you apply the accompanying .hg patch to sage-main?

nope, but that fixed the problem. It didn't seem obvious, so I ignored
that bundle.

With the old Cython:

==26784== LEAK SUMMARY:
==26784==    definitely lost: 0 bytes in 0 blocks.
==26784==      possibly lost: 251,078 bytes in 625 blocks.
==26784==    still reachable: 36,000,918 bytes in 20,368 blocks.
==26784==         suppressed: 0 bytes in 0 blocks.

With the new Cython and  n=1 I get:

==31741== LEAK SUMMARY:
==31741==    definitely lost: 0 bytes in 0 blocks.
==31741==      possibly lost: 253,574 bytes in 633 blocks.
==31741==    still reachable: 36,244,988 bytes in 20,359 blocks.
==31741==         suppressed: 0 bytes in 0 blocks.

So there has been a slight decrease in the number of blocks that are
still reachable or possibly lost, even thought the amount of memory
increased. I will try with n=2 to see if anything changes. How do I
change the default by the way?

Cheers,

Michael


--~--~---------~--~----~------------~-------~--~----~
To post to this group, send email to sage-devel@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at http://groups.google.com/group/sage-devel
URLs: http://sage.scipy.org/sage/ and http://modular.math.washington.edu/sage/
-~----------~----~----~----~------~----~------~--~---

Reply via email to