#4425: [with patch, positive review] sqrt(4) returns a SymbolicComposition 
instead
of the number 2!
------------------------------+---------------------------------------------
 Reporter:  was               |        Owner:  kcrisman
     Type:  defect            |       Status:  new     
 Priority:  major             |    Milestone:  sage-3.2
Component:  basic arithmetic  |   Resolution:          
 Keywords:                    |  
------------------------------+---------------------------------------------
Changes (by was):

  * summary:  [with patch, needs review] sqrt(4) returns a
              SymbolicComposition instead of the number 2! =>
              [with patch, positive review] sqrt(4) returns a
              SymbolicComposition instead of the number 2!

Comment:

 I attached a trivial followup patch to sqrt-nonsymbolic-1.patch which adds
 a docstring for the _do_sqrt function (which had no docstring) and some
 doctests to that function.  The new docstring also clarifies that the
 extend option to _do_sqrt is completely ignored, and why that is the right
 behavior.

 The original patch sqrt-nonsymbolic-1.patch and that patch plus the
 attached followup sqrt-nonsymblic-2.patch together pass all doctests for
 me in rings and calculus.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4425#comment:5>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of 
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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