#5828: number fields -- serious bug in coercion to a relative extension of 
degree
1
-----------------------------------------+----------------------------------
   Reporter:  was                        |       Owner:  davidloeffler
       Type:  defect                     |      Status:  needs_work   
   Priority:  minor                      |   Milestone:  sage-4.5.2   
  Component:  number fields              |    Keywords:               
     Author:  Luis Felipe Tabera Alonso  |    Upstream:  N/A          
   Reviewer:  Marco Streng               |      Merged:               
Work_issues:  reST formatting            |  
-----------------------------------------+----------------------------------
Changes (by mstreng):

 * cc: mstreng (added)
  * priority:  critical => minor
  * status:  needs_review => needs_work
  * work_issues:  => reST formatting
  * reviewer:  => Marco Streng


Comment:

 The tests should be indented, and you should only have "::" only on the
 last line before the test.
 The patch now has
 {{{
         TESTS::

         Check that #5828 is solved::

         sage: K.<w> = QuadraticField(-1)
         sage: KX.<X> = K[]
         sage: H.<h> = K.extension(X-1)
         sage: H(w)
         w
 }}}
 but I think it should be
 {{{
         TESTS:

         Check that #5828 is solved::

             sage: K.<w> = QuadraticField(-1)
             sage: KX.<X> = K[]
             sage: H.<h> = K.extension(X-1)
             sage: H(w)
             w
 }}}

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5828#comment:4>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to