Error in preparsing generators, QuadraticField

[ncalexan]

sage: K.<a> = QuadraticField(-55, 'a')
---------------------------------------------------------------------------
<type 'exceptions.TypeError'>             Traceback (most recent call last)

/Users/ncalexan/emacs/<ipython console> in <module>()

<type 'exceptions.TypeError'>: QuadraticField() got multiple values for keyword argument 'names'

Component: number fields

Keywords: preparse generators QuadraticField

Author: Luis Felipe Tabera Alonso

Reviewer: Nick Alexander

Merged: sage-4.6.alpha2

Issue created by migration from https://trac.sagemath.org/ticket/1135

[loefflerd]

comment:2

This clearly belongs in "number fields" ("interfaces" is for Sage interfaces to other software, not for user-interface issues).

[lftabera]

Attachment: trac-1135.patch.gz

[lftabera]

comment:3

This is a small error that is embarrasingly old. This patch solves the problem. You can use a generator name and the preparser in all combinations.

• I have added a default name for the generator 'a' to be consistent with NumberField.

• I have documented the behaviour of Sage when QuadraticField and NumberField are given two generators but there is a conflict in their names. The generator name given by the preparser has precedence in this case.

[lftabera]

Author: Luis Felipe Tabera Alonso

[ncalexan]

comment:4

Looks fine to me and passes tests on sage.math. Positive review!