Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fix %autopx magic #349

Closed
wants to merge 4 commits into
from
Commits on Apr 11, 2011
  1. fix %autopx in scripts by calling run_code for each ast node

    minrk committed Apr 9, 2011
    Previously, there was no call to override if %autopx was called inside
    a single cell (e.g. %run script.ipy). This splits run_ast_nodes, so that
    each node gets a separate call to run_code.
    
    reviewed changes:
    * remove old use of 'new'
    * fix interactiveshell.run_code docstring
    
    Additional change:
    automatically load parallelmagic extension on view.activate()
  2. add tests for parallel magics

    minrk committed Apr 10, 2011
    tests revealed some changes that should be made:
    
    %px also prints remote stdout in blocking mode,
    and actually raises the RemoteError (not just display)
    if there is one
  3. remove code_to_run attribute, updates per review.

    minrk committed Apr 11, 2011
    review changes:
    
    * change 1/0 to True/False in run_code
    * remove interactiveshell.code_to_run
    * remove superfluous if-tests checking for empty lists in run_ast_nodes
    
    closes gh-349