#5404: [with patch, needs work] deprecate numerical_sqrt
----------------------+-----------------------------------------------------
Reporter: robertwb | Owner: burcin
Type: defect | Status: new
Priority: minor | Milestone: sage-4.0
Component: calculus | Keywords:
----------------------+-----------------------------------------------------
Comment(by jhpalmieri):
The deprecation message doesn't make sense to me:
{{{
sage: sqrt(10.1, prec=5)
---------------------------------------------------------------------------
TypeError Traceback (most recent call
last)
...
TypeError: sqrt() got an unexpected keyword argument 'prec'
}}}
I think the issue here is that in real_mpfr.pyx, the sqrt method doesn't
accept a 'prec' argument. If I do {{{search_def('sqrt')}}}, I see lots of
sqrt methods without a prec argument. Perhaps the sqrt function in
calculus/calculus.py should check for a {{{TypeError}}} in addition to an
{{{AttributeError}}}? In any case, we can't give a deprecation message
which suggests using code which doesn't work.
We also need some doctests in the sqrt function in calculus/calculus.py
using all of the advertised arguments.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5404#comment:3>
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
-~----------~----~----~----~------~----~------~--~---
