Skip to content

Block on the window itself rather than the application's event loop - #14179

Merged
larsoner merged 2 commits into
mne-tools:mainfrom
larsoner:block
Aug 20, 2026
Merged

Block on the window itself rather than the application's event loop#14179
larsoner merged 2 commits into
mne-tools:mainfrom
larsoner:block

Conversation

@larsoner

Copy link
Copy Markdown
Member

Draft to help with mne-tools/mne-qt-browser#448 (and make it block better anyway IMO -- only block until the window of interest is closed rather than all possible windows).

SIGINT gymnastics adapted from matplotlib. Code drafted by Claude Opus 5 and iterated / reviewed by me.

@larsoner
larsoner force-pushed the block branch 2 times, most recently from 69c5c6b to 5dbc19c Compare August 20, 2026 18:23
@larsoner
larsoner marked this pull request as ready for review August 20, 2026 21:15
@larsoner
larsoner requested a review from drammock as a code owner August 20, 2026 21:15
@larsoner
larsoner requested a review from agramfort as a code owner August 20, 2026 21:28
@larsoner
larsoner enabled auto-merge (squash) August 20, 2026 21:29
@larsoner
larsoner merged commit 87d6459 into mne-tools:main Aug 20, 2026
39 of 40 checks passed
@larsoner
larsoner deleted the block branch August 20, 2026 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant