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

FIX: Add version gate to GTK4 calls when necessary#29363

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
greglucas merged 1 commit intomatplotlib:mainfromgreglucas:gtk4-guievent
Jan 6, 2025

Conversation

greglucas
Copy link
Contributor

PR summary

get_current_event() calls within pygobject would cause segfaults. This was fixed in 3.47, so gate the guiEvent information to only access that method in versions that don't segfault.

closes#29350

PR checklist

@greglucasgreglucas added this to thev3.10.1 milestoneDec 20, 2024
@tacaswelltacaswell added the Release criticalFor bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions. labelDec 20, 2024
@@ -37,6 +39,7 @@ class FigureCanvasGTK4(_FigureCanvasGTK, Gtk.DrawingArea):

def __init__(self, figure=None):
super().__init__(figure=figure)
print("GI version", gi.version_info)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Suggested change
print("GI version", gi.version_info)

stray debugging print.

Copy link
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Thanks! I guess this confirms I was testing it locally with various gi versions ;) and this did work on both lower and upper

tacaswell reacted with laugh emoji
get_current_event() calls within pygobject would cause segfaults. Thiswas fixed in 3.47, so gate the guiEvent information to only accessthat method in versions that don't segfault.
Copy link
Member

@timhoffmtimhoffm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Mypy failure is unrelated (#29362)

@tacaswell
Copy link
Member

"power-cycled" to re-run against current main which should fix the mypy issues.

@greglucasgreglucas merged commitcb2d928 intomatplotlib:mainJan 6, 2025
61 of 63 checks passed
@greglucasgreglucas deleted the gtk4-guievent branchJanuary 6, 2025 16:12
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull requestJan 6, 2025
rcomer added a commit that referenced this pull requestJan 6, 2025
…363-on-v3.10.xBackport PR#29363 on branch v3.10.x (FIX: Add version gate to GTK4 calls when necessary)
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@tacaswelltacaswelltacaswell approved these changes

@timhoffmtimhoffmtimhoffm approved these changes

Assignees
No one assigned
Labels
GUI: gtkRelease criticalFor bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions.
Projects
None yet
Milestone
v3.10.1
Development

Successfully merging this pull request may close these issues.

[Bug]: Matplotlib causes segmentation fault when hovering mouse over graph
3 participants
@greglucas@tacaswell@timhoffm

[8]ページ先頭

©2009-2025 Movatter.jp