-
Notifications
You must be signed in to change notification settings - Fork 57
update pygfx pin #939
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update pygfx pin #939
Conversation
|
Basically make sure that imgui keeps working. Longer answer: we made changes in the latest rendercanvas and wgpu. The changes to rendercanvas were meant to be compatible with earlier versions of wgpu, but they were not for the imgui adapter. So wgpu-py was updated to fix that, but pygfx 0.15.0 has an upper bound on wgpu, so 😕 |
|
Doesn't seem to be related to imgui, the non imgui regen screenshots is also failing with nans 🤔. Any chance you could take a look? |
|
Odd, it does not seem to be a problem for PyGfx ... I have zeroes in on the problem - it seems related to some of the size-refactoring applied recently in rendercanvas. It looks like I need to go now, but will work on a solution this evening / tomorrow. |
|
Ok, so rendercanvas kindof assumes that there will be no rendering outside of a draw event. But the logic to take screenshots does do that. The recent refactoring in rendercanvas apparently made is less resilient to that. Let's fix that in both places; make FPL not render outside of a draw event, and make rendercanvas resilient for when it does happen. Remaining errors: Plus the doc-gallery errors; these actually also happen at the Pygfx repo. |
|
📚 Docs preview built and uploaded! https://www.fastplotlib.org/ver/update-pygfx-pin |
|
Looks like the upstream fixes did the trick! |
|
@clewis7 gtg! Thanks @almarklein !! 🥳 |
@almarklein curious what exactly the new lower bounds on wgpu and rendercanvas do