#10709: speed up matrix actions on Manin symbols
-----------------------------+----------------------------------------------
Reporter: AlexGhitza | Owner: craigcitro
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-4.7
Component: modular forms | Keywords: manin symbol binomial
Author: | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
-----------------------------+----------------------------------------------
Changes (by AlexGhitza):
* status: needs_work => needs_review
Comment:
Replying to [comment:2 davidloeffler]:
> I don't like the following:
> {{{
> sage: a = ZZ(10); b = (2*a).binomial(a, algorithm="spam")
> sage: b is None
> True
> }}}
> It should probably raise an error if the algorithm is unrecognized, not
just fall of the end of the function and return None.
Done.
> Also, the "algorithm = mpir" option doesn't seem to allow the
computation to be interrupted with Ctrl-C, so if you feed it a very large
number by mistake you end up with no alternative but to kill your entire
sage session. (The Pari implementation seems to handle interrupts better.)
Fixed.
> Finally, I'd very much prefer it if the generic "binomial" function,
called with an integer object, just called the binomial method of that
object, and allowed the user to specify an algorithm there instead.
Done.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10709#comment:4>
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.
