#7581: use prCopyR to coerce multivariate polynomials in the simple case
-----------------------------------+----------------------------------------
Reporter: malb | Owner: malb
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-4.3
Component: commutative algebra | Keywords:
Work_issues: | Author: Martin Albrecht
Upstream: N/A | Reviewer:
Merged: |
-----------------------------------+----------------------------------------
Changes (by SimonKing):
* status: needs_work => needs_review
Comment:
{{{sage -testall}}} passes.
Your patch provides a strong improvement. However, genuine Singular is
match faster:
{{{
> system("--ticks-per-sec",1000);
> ring r1 = 0, (x(1..10000)),dp;
> poly p = x(10000);
> ring r2 = 0, (x(1..10001)), dp;
> int t = timer;
> poly q1 = imap(r1,p);
> timer-t;
410
> poly q2 = fetch(r1,p);
> timer-t;
430
}}}
So, 0.41s for imap and only 0.02s for fetch, in a situation where (even
with your patch) Sage needs 5.43s! And I think imap does exactly what one
needs in element conversion -- I mean, take a variable from one ring and
return the variable with the same name in another ring.
Is it possible to wrap imap in libsingular? This would probably yield a
drastic improvement (factor 10?).
But I guess this should be a different ticket, and since your patch does
provide a good improvement, I give it a positive review.
There is no button for "positive review"? Well, I guess I first have to
change it into "needs review", and then push the button...
Cheers,
Simon
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7581#comment:12>
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.
