Re: [sage-devel] Re: Memory leak with matrices and finite fields

2021-05-14 Thread 'Jonathan Kliem' via sage-devel
Its the same code in the Main line. Sorry for the confusion. I googled it and that is how I found the exact place where this is defined. What I linked is how it looks now accounting for changes in gmp 6.2. Flint 2.6.3 has it but not 2.5.2.On May 14, 2021 10:31 PM, Dima Pasechnik wrote: > > On

Re: [sage-devel] Re: Memory leak with matrices and finite fields

2021-05-14 Thread Dima Pasechnik
On Fri, May 14, 2021 at 4:03 PM 'jonatha...@googlemail.com' via sage-devel wrote: > > I think I tracked it down. > > https://github.com/BrianGladman/flint/blob/trunk/gmpcompat.h this is a fork, dealing with Windows support of Flint. We should be working with https://github.com/wbhart/flint2 whic

Re: [sage-devel] Re: Memory leak with matrices and finite fields

2021-05-14 Thread 'Jonathan Kliem' via sage-devel
Thanks Freddie for reporting this. I'm glad it resolves so easily. Yes, that is a mistake in our 9.2 configure. One can fix this by passing `--with-system-gmp=no` to configure, but it should have rejected gmp 6.2 as long as flint 2.5.2 is the install candidate. So you can update sage or rebu

Re: [sage-devel] Re: Memory leak with matrices and finite fields

2021-05-14 Thread 'Freddie Manners' via sage-devel
Awesome, many thanks for running this down. `sage --standard | grep gmp` >> gmp.6.1.2 (not_installed) `equery --quiet list gmp` >> dev-libs/gmp-6.2.1-r1 `sage --standard | grep flint` >> flint...2.5.2.p5 (2.5.2.p5) So yes, that

Re: [sage-devel] Re: Memory leak with matrices and finite fields

2021-05-14 Thread 'jonatha...@googlemail.com' via sage-devel
I think I tracked it down. https://github.com/BrianGladman/flint/blob/trunk/gmpcompat.h In there you see that `flint_mpz_set_si` is modified to work with GMP 6.2 I'm assuming this one is picked up from your system? `sage --standard | grep gmp` This code is not present yet in flint 2.5.2, which

Re: [sage-devel] Re: Memory leak with matrices and finite fields

2021-05-07 Thread 'Freddie Manners' via sage-devel
Dear all, Sorry for the delay getting back to you -- this was waiting in the moderation queue for a bit (me being new) and then I dropped the ball. > run the underlying functions > > M = identity_matrix(n) > L = M.list() > MS = sage.matrix.matrix_space.MatrixSpace(GF(101), 1000, 1000, sparse=F

Re: [sage-devel] Re: Memory leak with matrices and finite fields

2021-05-05 Thread Dima Pasechnik
we've fixed a memory leak in matrices recently (this fix will be in 9.3), not sure if this is related https://trac.sagemath.org/ticket/31340 On Wed, May 5, 2021 at 8:45 AM 'jonatha...@googlemail.com' via sage-devel wrote: > > Dear Freddie, > > thank you for the report. > > Fortunatly or unfortuna

[sage-devel] Re: Memory leak with matrices and finite fields

2021-05-05 Thread 'jonatha...@googlemail.com' via sage-devel
Dear Freddie, thank you for the report. Fortunatly or unfortunately, I can't reproduce this on sage 9.2 or on the develop branch (on ubuntu focal with libflint-2.5.2 and debian buster with sage's flint 2.6.3). This might be a bug with flint. What flint are you using? You can go into detail, t