#3979: Power series composition messes up precision
--------------------------------------------------------+-------------------
Reporter: kedlaya | Owner:
somebody
Type: defect | Status:
needs_work
Priority: critical | Milestone:
sage-4.7.2
Component: algebra | Keywords:
power series, composition, precision
Work_issues: | Upstream: N/A
Reviewer: Kiran Kedlaya, Luis Felipe Tabera Alonso | Author:
Francis Clarke
Merged: | Dependencies:
--------------------------------------------------------+-------------------
Comment(by fwclarke):
Replying to [comment:8 lftabera]:
Thanks for your careful look at the patch. Most of the problems can be
fixed quite easily, though the one at line 936 in `power_series_poly`
could be more difficult. I hope to submit a new patch soon.
> On file power_series_mpoly @74 Add documentation and a valid example to
the __call__ method. Each method or function that is modified need to
have a correct documentation and doctest.
The reason I removed rather than corrected the doctest in
`power_series_mpoly` was that it does '''not''' test this `__call__`
function but the one in `power_series_poly` (I've seen too many such
doctests). In fact I'm not sure that this file is used at all. In fact
the first line is
{{{
# NOT ready to be used -- possibly should be deleted.
}}}
However I didn't have the confidence to delete it myself, and add other
issues to an already complicated patch.
Adding documentation to this function would be hard to do since the whole
file is so poorly documented that I can't understand what it's for.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/3979#comment:9>
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.
