#7958: Conversion of rationals into the fraction field of integer polynomials
-------------------------+--------------------------------------------------
   Reporter:  spancratz  |       Owner:  robertwb  
       Type:  defect     |      Status:  needs_work
   Priority:  minor      |   Milestone:  sage-4.3.2
  Component:  coercion   |    Keywords:            
Work_issues:             |      Author:  spancratz 
   Upstream:  N/A        |    Reviewer:            
     Merged:             |  
-------------------------+--------------------------------------------------
Changes (by wjp):

  * status:  needs_review => needs_work


Comment:

 Your fix does work great for QQ, but this is actually a more general issue
 than just QQ:

 {{{
 sage: _.<x> = ZZ[]
 sage: K.<a> = NumberField(x^2-2)
 sage: R.<b> = K.ring_of_integers()
 sage: S.<y> = R[]
 sage: F = FractionField(S)
 sage: F(1)/F(a)
 1/a
 sage: F(1/a)
 *boom*
 }}}

 And a minor issue: I think the comment about QQ should be a code comment
 rather than in the doctest, since it might now confuse users (who might
 think they need to handle QQ specially themselves).

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7958#comment:3>
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