#10496: The default call method of maps does not do as promised by the
documentation
-------------------------------+--------------------------------------------
Reporter: SimonKing | Owner: robertwb
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.6.2
Component: coercion | Keywords: generic call map
Author: Simon King | Upstream: N/A
Reviewer: David Roe | Merged:
Work_issues: speed things up |
-------------------------------+--------------------------------------------
Changes (by SimonKing):
* status: needs_work => needs_review
Comment:
Here are the same tests again, with the updated patch:
{{{
sage: R.<x,y> = QQ[]; phi=R.hom([y,x])
sage: timeit("a=phi(y)")
625 loops, best of 3: 33 µs per loop
sage: timeit("a=phi(5)")
625 loops, best of 3: 39.2 µs per loop
sage: D={(0, 2): -1, (0, 0): -1, (1, 1): 7, (2, 0): 1/3}
sage: phi(D)
-x^2 + 7*x*y + 1/3*y^2 - 1
sage: timeit("a=phi(D)")
625 loops, best of 3: 121 µs per loop
sage: I = R*[x^2*y^4-2*y^2, 3*x*y-y^3]
sage: phi(I)
Ideal (x^4*y^2 - 2*x^2, -x^3 + 3*x*y) of Multivariate Polynomial Ring in
x, y over Rational Field
sage: timeit("a=phi(I)")
625 loops, best of 3: 163 µs per loop
}}}
I think that are acceptable timings: No significant slowdown, but a
significant speedup in one case, and some bug fixes. Back to "needs
review", modulo passing doctests...
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10496#comment:6>
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.
