[sage-devel] Re: Bug in FreeQuadraticModule.discriminant

2016-08-31 Thread Simon Brandhorst
Sorry for posting twice. Got confused after the first time since the post did not appear right away. I can do the next step and open a ticket. Though first I have to get the source code of sage compiled... (was working with binaries before) I'll post the log file in another thread. On Wednesd

Re: [sage-devel] Re: Bug in FreeQuadraticModule.discriminant()

2016-08-31 Thread John Cremona
On 31 August 2016 at 16:24, leif wrote: > John Cremona wrote: > > Indeed, that is a bug in line 371 of the file > > sage/modules/free_quadratic_module.py -- an easy fix. > > ... and more than 8 years old. > > The check whether the rank is even or not is useless as well; '//' in > Python is intege

[sage-devel] Re: Bug in FreeQuadraticModule.discriminant()

2016-08-31 Thread leif
John Cremona wrote: > Indeed, that is a bug in line 371 of the file > sage/modules/free_quadratic_module.py -- an easy fix. ... and more than 8 years old. The check whether the rank is even or not is useless as well; '//' in Python is integer division (taking the floor). (And the docstrings don'

[sage-devel] Re: Bug in FreeQuadraticModule.discriminant

2016-08-31 Thread Frédéric Chapoton
yes, indeed this is a bug. Thanks for reporting. Would you like to make the next step to solve that ? Then you should ask for an account on trac.sagemath.org and open a ticket. Instructions are available here: http://www.sagemath.org/git-developer-guide/ Frederic Le mercredi 31 août 2016 15:3