#10709: speed up matrix actions on Manin symbols
-----------------------------+----------------------------------------------
   Reporter:  AlexGhitza     |       Owner:  craigcitro           
       Type:  enhancement    |      Status:  needs_work           
   Priority:  major          |   Milestone:  sage-4.7             
  Component:  modular forms  |    Keywords:  manin symbol binomial
     Author:                 |    Upstream:  N/A                  
   Reviewer:                 |      Merged:                       
Work_issues:                 |  
-----------------------------+----------------------------------------------
Changes (by davidloeffler):

  * status:  needs_review => needs_work


Comment:

 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.

 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.)

 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.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10709#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.

Reply via email to