#13218: Laurent series ring doesn't carry over default prec
-----------------------------+----------------------------------------------
Reporter: roed | Owner: AlexGhitza
Type: defect | Status: needs_work
Priority: major | Milestone: sage-5.2
Component: algebra | Resolution:
Keywords: | Work issues:
Report Upstream: N/A | Reviewers:
Authors: David Roe | Merged in:
Dependencies: | Stopgaps:
-----------------------------+----------------------------------------------
Changes (by robertwb):
* status: needs_review => needs_work
Comment:
The example in change_ring could be more illustrative, though the test you
have there is useful as well.
Line 144-146, one space extra indentation.
Bug in is_field, namely
{{{
sage: LaurentSeriesRing(ZZ, 't').is_field()
True
}}}
In {{{_repr_}}}, test a sparse ring as well.
It'd be nice to show at least one example (and non-example?) of each of
the bullet points in _coerce_impl.
All of these points are fairly minor, but we might as well attack them
while we're here. Other than that, looks good.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13218#comment:2>
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.
