- Notifications
You must be signed in to change notification settings - Fork5.5k
Upgrade xterm.js to 2.9.2#2849
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.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
rgbkrk commentedSep 18, 2017
Does this fix#2850 for you? |
takluyver commentedSep 19, 2017
It looks like the fix for that issue is still waiting in a PR on xterm. I've restarted one test job that failed; if it passes next time, I'm happy to merge this. |
takluyver commentedSep 19, 2017
@Carreau it looks like the notebook tests rely on passing a base64-encoded unicode string to Do you think we should change the notebook tests to not rely on that, or put back the ability to pass base64 encoded data into the class in IPython? |
rgbkrk commentedSep 19, 2017
Change the notebook test |
minrk commentedSep 19, 2017
#2851 should update the tests to use bytes instead. |
minrk commentedSep 19, 2017 • edited by rgbkrk
Loading Uh oh!
There was an error while loading.Please reload this page.
edited by rgbkrk
Uh oh!
There was an error while loading.Please reload this page.
If there is a fix to be made on the IPython side, I think it would be to make the ifisinstance(data,unicode):# already base64b64data=dataelse:b64data=b2a_base64(data).decode('ascii') |
takluyver commentedSep 20, 2017
Reopening to rerun the tests now that#2851 is merged. |
No description provided.