#11287: Interface to runsnake and import_statements
------------------------------------------+---------------------------------
   Reporter:  nthiery                     |          Owner:  tbd                
              
       Type:  enhancement                 |         Status:  needs_review       
              
   Priority:  major                       |      Milestone:  sage-4.7.1         
              
  Component:  performance                 |       Keywords:  runsnake, prun, 
profiling, days30
Work_issues:                              |       Upstream:  N/A                
              
   Reviewer:  Franco Saliola, Simon King  |         Author:  Nicolas M. ThiƩry  
              
     Merged:                              |   Dependencies:                     
              
------------------------------------------+---------------------------------

Comment(by stumpc5):

 Hi, I pushed a review concerning two minor things I ran into:

 on the sage command line, spaces do not cause trouble, but the profiler
 doesn't like them:
 {{{
 sage: runsnake(' 3')
 ...
 IndentationError: unexpected indent (<string>, line 1)
 }}}

 Now, spaces are deleted (or does anyone see problems occurring from
 that?).

 The second is that sage accepts "^", while python doesn't, so these are
 replaced by "**".
 {{{
 sage: runsnake('QQ(3)^2')
 ...
 RuntimeError: Use ** for exponentiation, not '^', which means xor
 in Python, and has the wrong precedence.
 }}}

 You can fold the changes if you like, or remove them otherwise...

 Btw: the patch doesn't apply on a plain 4.6.2.

 Best, Christian

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