On Nov 24, 6:26 am, Bill Hart <[EMAIL PROTECTED]> wrote:
> No, you are correct. The tests spend many minutes doing
> test_fmpz_poly_div_newton. It's supposed to take 3s!
Yep, the first time I thought I had hung itself and I rebuild from
scratch because I had that 6 minute figure in mmy head.
>The problem is
> because of the memory leak in _fmpz_poly_reverse. Perhaps we better
> fix this before people do any serious testing if that is OK. I hadn't
> noticed it since I test with the debugging memory manager on, which
> ironically doesn't pick up memory leaks and stops this problem from
> manifesting.
>
> To fix the problem, simply insert the following line between lines
> 1485 and 1486 of fmpz_poly.c (i.e. make it the third last line of the
> function _fmpz_poly_reverse):
Will do.
>
> flint_stack_release();
See, automated testing benefits everybody ;)
>
> Bill.
Cheers,
Michael
>
> On 24 Nov, 04:26, Bill Hart <[EMAIL PROTECTED]> wrote:
>
> > Incidentally, the tests take 6 mins on sage.math for me. I suspect
> > something is wrong. Is it linking against the right GMP?
>
> > Bill.
>
> > On 24 Nov, 03:46, mabshoff <[EMAIL PROTECTED]
>
> > dortmund.de> wrote:
> > > Hello folks,
>
> > > Bill has fixed a couple of bugs in flint since r1072 that were corner
> > > cases that only happened on Core Duos, so I have updated the spkg to
> > > r1075. It is available at
>
> > >http://sage.math.washington.edu/home/mabshoff/flint-0.9-r1075.spkg
>
> > > It runs the spkg-check script per default - this will be deactivated
> > > in the final 2.8.14 release. We do this because Bill Hart needs
> > > feedback and extended testing for FLINT that the Sage doctests cannot
> > > provide. The tests take about 15-30 minutes to run (15 on sage.math,
> > > 30 in my laptop). Please report back whether the tests failed or
> > > succeeded, together with info about CPU, operating system and
> > > compiler. Bill: is there a standard format you would like?
>
> > > Cheers,
>
> > > Michael
>
> > > One more thing: Bill, when I checked out trunk, i.e. r1086 compilation
> > > fails with:
>
> > > gcc -std=c99 -I/tmp/Work-mabshoff/release-cycles-2.8.14/
> > > sage-2.8.14.rc0/local/include/ -I/tmp/Work-mabshoff/release-
> > > cycles-2.8.14/sage-2.8.14.rc0/local/include -I/tmp/Work-mabshoff/
> > > release-cycles-2.8.14/sage-2.8.14.rc0/local/include -funroll-loops -
> > > fexpensive-optimizations -mtune=opteron -march=opteron -fPIC -funroll-
> > > loops -O3 -o delta_qexp delta_qexp.o mpn_extras.o mpz_extras.o memory-
> > > manager.o ZmodF.o ZmodF_mul.o ZmodF_mul-tuning.o fmpz.o fmpz_poly.o
> > > mpz_poly-tuning.o mpz_poly.o ZmodF_poly.o long_extras.o -L/tmp/Work-
> > > mabshoff/release-cycles-2.8.14/sage-2.8.14.rc0/local/lib/ -L/tmp/Work-
> > > mabshoff/release-cycles-2.8.14/sage-2.8.14.rc0/local/lib/ -L/tmp/Work-
> > > mabshoff/release-cycles-2.8.14/sage-2.8.14.rc0/local/include -lgmp -
> > > lpthread -lm -lntl
> > > make: *** No rule to make target `BLTcubes.c', needed by `BLTcubes'.
> > > Stop.
> > > Error building flint shared library.
>
> > > Since there are some modifications to the makefiles in the spkg (it
> > > seems that we add targets for OSX) we might want to merge those
> > > changes back into trunk. I will check and ping the person who wrote
> > > the makefile.
--~--~---------~--~----~------------~-------~--~----~
To post to this group, send email to [email protected]
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/
-~----------~----~----~----~------~----~------~--~---
