Skip to content
New issue

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

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

iPython notebook printing output to terminal instead of screen #43

Closed
dandanxu opened this issue Oct 30, 2014 · 4 comments
Closed

iPython notebook printing output to terminal instead of screen #43

dandanxu opened this issue Oct 30, 2014 · 4 comments
Labels
bug P2:normal

Comments

@dandanxu
Copy link
Member

screen shot 2014-10-30 at 5 02 08 pm

ipython notebook is supposed to print to the notebook/screen, but after importing solvebio, stops doing so, and prints to the terminal that ipython notebook opened in.

it's due to these lines:
https://github.com/solvebio/solvebio-python/blob/master/solvebio/utils/printing.py#L10-14
@davecap
Copy link
Member

davecap commented Oct 30, 2014

This issue is specifically related to the reload(sys) in printing.py.
I wonder if this affects usage in Terminal.com demos.

@rocky
Copy link
Contributor

rocky commented Oct 30, 2014

Might be related to issue #21

@dandanxu
Copy link
Member Author

it does affect the terminal.com demos - it makes it so that you can't print the output anywhere haha. The terminals.com demo is actually the reason I noticed, I didn't realize it was a bug before (I thought it was just normal ipython notebooks behaviour).

rocky pushed a commit that referenced this issue Oct 31, 2014
Save and restore sys filehandles before reload(sys). FIxues issues #43 …
cb97d97 
…and #21 possibly.

Use LINES and COLUMNS environment variables if we can't get it from stty.
davecap added a commit that referenced this issue Oct 31, 2014
@davecap
Merge pull request #46 from solvebio/feature/ipython-notebook-and-iss…
f3beaa5 
…ues-21-43

Address issue #43 and possibly #21
@rocky
Copy link
Contributor

rocky commented Oct 31, 2014

Release 1.6.0 is out. If this solves the problem, please close this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug P2:normal
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@davecap @rocky @dandanxu