search

sagemath / sage

Main repository of SageMath. Now open for Issues and Pull Requests.
https://www.sagemath.org
Other
1.2k stars 413 forks source link

Make number_field .galois_closure require a name and .galois_conjugates take an explicit field. #2083

Closed 93749b3a-c0a4-47a7-b178-004ba08b0b97 closed 16 years ago

93749b3a-c0a4-47a7-b178-004ba08b0b97 commented 16 years ago

.galois_closure used to guess a variable name, which is not very Sage-like. This and a related issue with .galois_conjugates are fixed.

Component: number theory

Keywords: number field galois closure names

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

93749b3a-c0a4-47a7-b178-004ba08b0b97 commented 16 years ago

Attachment: ncalexan-nf-galois-closure.patch.gz

mwhansen commented 16 years ago
comment:2

Looks good to me.

85eec1a4-3d04-4b4d-b711-d4db03337c41 commented 16 years ago
comment:3

Merged in Sage 2.10.2.alpha0