Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implementing number fields defined by non-monic polynomials #4041

Closed
sagetrac-mabshoff mannequin opened this issue Sep 2, 2008 · 3 comments
Closed

Implementing number fields defined by non-monic polynomials #4041

sagetrac-mabshoff mannequin opened this issue Sep 2, 2008 · 3 comments

Comments

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Sep 2, 2008

Implementing non-monic number fields would be hard, but not if the
leading coefficient is a unit :)

sage: NumberField(x^2 - 2, 'a') 
Number Field in a with defining polynomial x^2 - 2 
sage: NumberField(-x^2 - 2, 'a') 
--------------------------------------------------------------------------- 
NotImplementedError                       Traceback (most recent call   
last) 
/Users/ncalexan/Devel/Squeak-3.10-1/platforms/unix/bld/<ipython   
console> in <module>() 
/Users/ncalexan/sage-3.0.6/local/lib/python2.5/site-packages/sage/ 
rings/number_field/number_field.py in NumberField(polynomial, name,   
check, names, cache) 
     288 
     289     if polynomial.degree() == 2: 
--> 290         K = NumberField_quadratic(polynomial, name, check) 
     291     else: 
     292         K = NumberField_absolute(polynomial, name, None, check) 
/Users/ncalexan/sage-3.0.6/local/lib/python2.5/site-packages/sage/ 
rings/number_field/number_field.py in __init__(self, polynomial, name,   
check) 
    6001             Number Field in a with defining polynomial x^2 - 4 
    6002         """ 
-> 6003         NumberField_absolute.__init__(self, polynomial,   
name=name, check=check) 
    6004         self._element_class =   
number_field_element_quadratic.NumberFieldElement_quadratic 
    6005         c, b, a = [rational.Rational(t) for t in   
self.defining_polynomial().list()] 
/Users/ncalexan/sage-3.0.6/local/lib/python2.5/site-packages/sage/ 
rings/number_field/number_field.py in __init__(self, polynomial, name,   
latex_name, check) 
    3272 
    3273     def __init__(self, polynomial, name, latex_name=None,   
check=True): 
-> 3274         NumberField_generic.__init__(self, polynomial, name,   
latex_name, check) 
    3275         self._element_class =   
number_field_element.NumberFieldElement_absolute 
    3276 
/Users/ncalexan/sage-3.0.6/local/lib/python2.5/site-packages/sage/ 
rings/number_field/number_field.py in __init__(self, polynomial, name,   
latex_name, check) 
     668                 raise TypeError, "polynomial must be defined   
over rational field" 
     669             if not polynomial.is_monic(): 
--> 670                 raise NotImplementedError, "number fields for   
non-monic polynomials not yet implemented." 
     671             if not polynomial.is_irreducible(): 
     672                 raise ValueError, "defining polynomial (%s)   
must be irreducible"%polynomial 
NotImplementedError: number fields for non-monic polynomials not yet   
implemented. 

Component: number theory

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

@sagetrac-mabshoff sagetrac-mabshoff mannequin added this to the sage-3.1.2 milestone Sep 2, 2008
@JohnCremona
Copy link
Member

comment:1

It's not clear to me exactly what Nick means: defining polys in Z[x] with leading coefficient -1, or something more general over Q, or also relative number fields.

@JohnCremona JohnCremona changed the title Implementing non-monic number fields Implementing number fields defined by non-monic polynomials Sep 2, 2008
@JohnCremona
Copy link
Member

comment:2

This ticket seems to be a duplicate of #252. So I think this one can be deleted.

@sagetrac-mabshoff
Copy link
Mannequin Author

sagetrac-mabshoff mannequin commented Sep 4, 2008

comment:3

As John pointed out this is a dupe of #252, so close this ticket.

Cheers,

Michael

@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Sep 4, 2008
@sagetrac-mabshoff sagetrac-mabshoff mannequin reopened this Sep 4, 2008
@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Sep 4, 2008
@sagetrac-mabshoff sagetrac-mabshoff mannequin removed this from the sage-3.1.2 milestone Sep 4, 2008
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants