| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
There was a problem hiding this comment.
I tested this out on a mac and this does get rid of the GIL warning and everything works as expected.
Also as an FYI, I tested this with multiple windows trying to resize/zoom/pan between them and all of that state seemed good to me.
I also ran the threading example from here: #5675 (comment), updating data in background threads and that also played nicely with resizing/interactiveness and the free-threaded build. So I think we are at least OK with reasonable expectations for now. It will be good to get people hammering this a bit more and more exotic use-cases to be tested out.
edit: I also just added constrained_layout to test our interaction with kiwisolver's free threaded capability and that works nicely too.
Sorry, something went wrong.
|
@greglucas just noticing that I'm hitting the issue fixed by this on a fresh Python 3.14.0t environment. And indeed, the latest release is 3.10.7, so this fix isn't included. Does it maybe make sense to backport? |
Sorry, something went wrong.
|
This does not feel like a back-port type thing to me, but could be talked into it. |
Sorry, something went wrong.
|
I would generally agree that this seems potentially odd to put in a bugfix release. On the other hand, this is pretty niche still and I'm not sure guarantees us to anything as it is pretty heavily opt-in at this point still. It has been 10 months since our last release https://github.com/matplotlib/matplotlib/releases/tag/v3.10.0, so I think people in the community are getting excited about updates and new features and wondering when they will be released, so I do understand the desire to get the features people have added out in some kind of release. I'm not sure if another bugfix release will happen again before the next version's release though, so maybe a backport is moot here 🤷 |
Sorry, something went wrong.
|
Actually, I do think this is valid for a backport because it was a bug that we forgot to mark this specific module in the original PR adding this declaration, so in a sense this is fixing that oversight. |
Sorry, something went wrong.
…backend extension
…tension (#30656) Co-authored-by: Greg Lucas <greg.m.lucas@gmail.com>
| Back | FazBrowse Home | New Git URL |
See #28611 (comment) for context.