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

Allow interrupts to be delivered once Python is fixed.#4006

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
tacaswell merged 1 commit intomatplotlib:masterfrommdehoon:allow_interrupts
Feb 9, 2015

Conversation

mdehoon
Copy link
Contributor

This is related to issue#3991. Until now the MacOSX backend did not deliver interrupts while the event loop is running as Python itself would ignore those interrupts anyway. This PR allows interrupts to be delivered once Python is fixed.

@tacaswell
Copy link
Member

@mdehoon Who else knows this code well enough to review it?

@tacaswelltacaswell added this to thev1.5.x milestoneJan 19, 2015
@tacaswell
Copy link
Member

and should this be back-ported to 1.4.x?

@mdehoon
Copy link
ContributorAuthor

@tacaswell Few people have a good understanding of event loops, in particular people coming from a scripting background. This PR is not a major change, so I think getting a review is not critical. I would suggest to wait until Python itself is fixed (because without that, this PR doesn't add any functionality anyway), and after that apply the PR and let some users test it. This PR does not need to be back-ported to 1.4.x.

@mdehoon
Copy link
ContributorAuthor

I have uploaded a patch for Python; see issue 23237 at
http://bugs.python.org/issue23237
With this patch, and this pull request, interrupts will be delivered for the MacOSX backend (and also for Tkinter). I would appreciate it if interested people can give this a try.

@tacaswell
Copy link
Member

Is there harm in merging this before python is fixed?

@mdehoon
Copy link
ContributorAuthor

This can be merged before Python is fixed, so it's OK to merge it now. This pull request also fixes issue#3654 .

tacaswell added a commit that referenced this pull requestFeb 9, 2015
BUG : Allow interrupts to be delivered once Python is fixed + path fixes
@tacaswelltacaswell merged commitee4f072 intomatplotlib:masterFeb 9, 2015
@mdehoonmdehoon deleted the allow_interrupts branchFebruary 9, 2015 02:52
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Projects
None yet
Milestone
v2.1
Development

Successfully merging this pull request may close these issues.

2 participants
@mdehoon@tacaswell

[8]ページ先頭

©2009-2025 Movatter.jp