#11780: Creating a polynomial ring over a number field results in a non-unique
polynomial ring over the rationals
-------------------------+--------------------------------------------------
   Reporter:  SimonKing  |          Owner:  robertwb                            
   
       Type:  defect     |         Status:  needs_review                        
   
   Priority:  major      |      Milestone:  sage-4.7.2                          
   
  Component:  coercion   |       Keywords:  non-unique polynomial ring number 
field
Work_issues:             |       Upstream:  N/A                                 
   
   Reviewer:             |         Author:  Simon King                          
   
     Merged:             |   Dependencies:                                      
   
-------------------------+--------------------------------------------------

Comment(by SimonKing):

 Replying to [comment:6 malb]:
 > Doctests pass. However, I gave the !TypeError thing another thought. If
 somebody changes the class for polynomials over the base field (prime
 field or rational field), couldn't that again lead to problems with non-
 identical parents? I.e. should we cause the !TypeError such that future
 developers will note there's a potential problem?

 Sorry, I saw your answer only now.

 Yes, on second thought, it seems reasonable to not catch the type error.

 I am sorry that our messages crossed (I have updated my patch, it now
 contains a second fix and one doctest). I will remove catching the type
 error in a minute.

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