#14146: Warn when creating number fields with non-monic-integral polynomials
----------------------------------------------------------------+-----------
       Reporter:  Bouillaguet                                   |         
Owner:  davidloeffler
           Type:  defect                                        |        
Status:  needs_review 
       Priority:  major                                         |     
Milestone:  sage-5.9     
      Component:  number fields                                 |    
Resolution:               
       Keywords:  pari, number field, monic, integral, integer  |   Work 
issues:               
Report Upstream:  N/A                                           |     
Reviewers:  Marco Streng 
        Authors:  Charles Bouillaguet                           |     Merged 
in:               
   Dependencies:                                                |      
Stopgaps:               
----------------------------------------------------------------+-----------

Old description:

> Add warnings for the problem at #252.

New description:

 Add warnings for the problem at #252.

 Apply:

 * [attachment:14146_number_fields.patch]
 * [attachment:14146_reviewer.patch]

--

Comment (by mstreng):

 I did some of the corrections myself that I did not get an answer to:

  * "..." instead of a doctest number
  * `NotImplementedError`, because it is more appropriate than `ValueError`
 and also more in line with the absolute case
  * import `warn` more directly

 However, I still don't understand why I get the warning printed twice:
 {{{
 sage: k.<a> = NumberField(x^2+1)
 sage: l.<b> = k.extension(x^3-1/2)
 /sage-5.6/local/lib/python2.7/site-
 packages/sage/rings/number_field/number_field_rel.py:307: UserWarning:
 PARI only handles integral absolute polynomials. Computations in this
 field might trigger PARI errors. See
 http://trac.sagemath.org/sage_trac/ticket/252
   warn("PARI only handles integral absolute polynomials. Computations in
 this field might trigger PARI errors. See
 http://trac.sagemath.org/sage_trac/ticket/252";)
 }}}

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14146#comment:15>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to