#12195: numerical_approx broken for vectors
------------------------------+---------------------------------------------
Reporter: robertwb | Owner: jason, was
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.8
Component: linear algebra | Keywords:
Work_issues: | Upstream: N/A
Reviewer: | Author:
Merged: | Dependencies:
------------------------------+---------------------------------------------
Comment(by rbeezer):
Robert,
Looks good, and very welcome. I should have noticed, and fixed, this
omission during my big push last year. Looks ready for a positive review,
but I have two questions.
1. Could this make the {{{.n()}}} method (immediate prior in source)
obsolete?
2. This does not seem to show up explicitly anywhere in the
documentation. Doing {{{v.N?}}} on an integer vector {{{v}}} seems to
just give the generic documentation for a numerical approximation to a
single scalar. For matrices, the approach is different. A bit more code,
and not overriding the scalar method: Plus: explicit documentation.
Minus: duplication of code, including a really bad approximation of {{{log
10}}} as discussed recently on sage-devel.
A suggestion - add a doctest in this patch for the generic method
exhibiting application to vectors. If that's agreeable, I'll make a
ticket to have matrices follow the model of this ticket (and maybe even do
it).
Rob
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12195#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.
