Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@kushalkolar
Copy link
Member

@almarklein curious what exactly the new lower bounds on wgpu and rendercanvas do

@kushalkolar kushalkolar requested a review from clewis7 as a code owner November 12, 2025 09:17
@almarklein
Copy link
Collaborator

almarklein commented Nov 12, 2025

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 😕

@kushalkolar
Copy link
Member Author

kushalkolar commented Nov 12, 2025

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?

@almarklein
Copy link
Collaborator

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 context.get_current_texture() is called before the canvas updated the size, so the returned texture is just one pixel.

I need to go now, but will work on a solution this evening / tomorrow.

@almarklein
Copy link
Collaborator

almarklein commented Nov 13, 2025

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:

=========================== short test summary info ============================
FAILED examples/notebooks/test_gc.ipynb::test_gc.ipynb - ValueError: extent x-range must be non-negative: [0. 0. 0. 0.]
FAILED examples/notebooks/quickstart.ipynb::quickstart.ipynb - ValueError: extent x-range must be non-negative: [0. 0. 0. 0.]

Plus the doc-gallery errors; these actually also happen at the Pygfx repo.

@github-actions
Copy link

github-actions bot commented Nov 14, 2025

📚 Docs preview built and uploaded! https://www.fastplotlib.org/ver/update-pygfx-pin

@almarklein
Copy link
Collaborator

Looks like the upstream fixes did the trick!

@kushalkolar
Copy link
Member Author

@clewis7 gtg!

Thanks @almarklein !! 🥳

@clewis7 clewis7 merged commit bd5101e into main Nov 14, 2025
36 checks passed
@clewis7 clewis7 deleted the update-pygfx-pin branch November 14, 2025 12:53
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.

4 participants