Merge lp://qastaging/~villemvainio/ipython/ipyrender-linuxfix into lp://qastaging/ipython/0.11
Proposed by
Ville M. Vainio
Status: | Work in progress |
---|---|
Proposed branch: | lp://qastaging/~villemvainio/ipython/ipyrender-linuxfix |
Merge into: | lp://qastaging/ipython/0.11 |
To merge this branch: | bzr merge lp://qastaging/~villemvainio/ipython/ipyrender-linuxfix |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Fernando Perez | Needs Fixing | ||
Brian Granger | none | Needs Fixing | |
Review via email:
|
To post a comment you must log in.
Unmerged revisions
- 1159. By Ville M. Vainio
-
only warn once
- 1158. By Ville M. Vainio
-
toclip: some simplification / optimization (only run 'which' once)
- 1157. By Ville M. Vainio
-
apply ipy_render from Matt Foster
Everything looks good, but I do have one question:
I see that you print a warning like this:
print "WARNING: ...."
Is there a reason we don't use the warnings module for this? I have seen this used elsewhere in IPython. Probably should change this code to use it as well unless there is a specific reason.