Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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

Merged
takluyver merged 1 commit intojupyter:masterfromcancan101:patch-1
Sep 20, 2017
Merged

Conversation

@cancan101
Copy link
Contributor

No description provided.

@rgbkrk
Copy link
Member

Does this fix#2850 for you?

@takluyvertakluyver added this to the5.2 milestoneSep 19, 2017
@takluyver
Copy link
Member

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
Copy link
Member

@Carreau it looks like the notebook tests rely on passing a base64-encoded unicode string toIPython.display.Image(), and the new release of IPython has broken that (the test only fails on Python 3, but I suspect it will display an invalid image with Python 2).

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
Copy link
Member

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?

Change the notebook test

@minrk
Copy link
Member

#2851 should update the tests to use bytes instead.

@minrk
Copy link
Member

minrk commentedSep 19, 2017
edited by rgbkrk
Loading

If there is a fix to be made on the IPython side, I think it would be to make theb64encode conditional on if the input data isbytes orunicode:

ifisinstance(data,unicode):# already base64b64data=dataelse:b64data=b2a_base64(data).decode('ascii')

@takluyver
Copy link
Member

Reopening to rerun the tests now that#2851 is merged.

@takluyvertakluyver reopened thisSep 20, 2017
@takluyvertakluyver merged commit39c27ac intojupyter:masterSep 20, 2017
@gnestorgnestor modified the milestone:5.2Oct 13, 2017
@github-actionsgithub-actionsbot locked asresolvedand limited conversation to collaboratorsApr 5, 2021
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

5.2

Development

Successfully merging this pull request may close these issues.

5 participants

@cancan101@rgbkrk@takluyver@minrk@gnestor

[8]ページ先頭

©2009-2025 Movatter.jp