#15345: Laurent polynomial rings don't preserve coercion
-------------------------------------+-------------------------------------
Reporter: darij | Owner:
Type: defect | Status: needs_review
Priority: major | Milestone: sage-6.2
Component: algebra | Resolution:
Keywords: laurent | Merged in:
polynomials, polynomials, | Reviewers: Travis Scrimshaw
coercion, categories | Work issues: add doctests
Authors: Frédéric | Commit:
Chapoton, David Roe | 8d541055054316406c45eb4ad44a068670bfe916
Report Upstream: N/A | Stopgaps:
Branch: |
u/tscrim/ticket/15345 |
Dependencies: |
-------------------------------------+-------------------------------------
Changes (by tscrim):
* reviewer: => Travis Scrimshaw
* branch: u/chapoton/15345 => u/tscrim/ticket/15345
* commit: 00393f332216d6d2b009842b02e1f9384ad0d807 =>
8d541055054316406c45eb4ad44a068670bfe916
Comment:
Okay, I've fixed up the Hall algebra documentation. I've changed
`to_fraction()` into `_fraction_pair()` since this is only suppose to be
used for the coercion (so it's hidden) and IMO it more accurately reflects
what is being returned. If you're happy with my changes, then up to
someone not likely what we're doing on sage-devel, it's a positive review
from me.
----
New commits:
||[http://git.sagemath.org/sage.git/commit/?id=f2e9d657fe345449be2d7479065f7dea36209607
f2e9d65]||{{{Fixed documentation for hall_algebra.py.}}}||
||[http://git.sagemath.org/sage.git/commit/?id=8d541055054316406c45eb4ad44a068670bfe916
8d54105]||{{{Renamed to_fraction to _fraction_pair.}}}||
--
Ticket URL: <http://trac.sagemath.org/ticket/15345#comment:25>
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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/groups/opt_out.
