Skip to content

Conversation

kushalkolar
Copy link
Member

my hunch is this is due to pygfx/pygfx#945

when set_data() is used, it wouldn't immediately update the camera related stuff which is why the axes don't appear on the initial draw after set_data(), as a user we don't notice it because that draw only lasts for a few milliseconds.

@kushalkolar kushalkolar requested a review from clewis7 as a code owner January 21, 2025 07:03
@kushalkolar
Copy link
Member Author

@clewis7 this should fix the current CI failure

Copy link

github-actions bot commented Jan 21, 2025

📚 Docs preview built and uploaded! https://www.fastplotlib.org/ver/fix-screenshot

@kushalkolar
Copy link
Member Author

I also changed the CI timeout from 10 mins to 15, for whatever reason it took llvmpipe etc. 7 mins to install. Maybe something changed on github actions 🤷‍♂️ . Anyways jobs usually finish in 5 mins, maybe stuff gets cached so if it hits a new machine or something it takes longer.

@kushalkolar kushalkolar mentioned this pull request Jan 21, 2025
@kushalkolar
Copy link
Member Author

idk what's taking so long with apt install on github actions lately, relaxed the timeout time to 25 mins 🤷‍♂️

@kushalkolar
Copy link
Member Author

wtf, so now all the jobs finished within 5mins?? Must be some caching things or apt servers were slow for some reason during those earlier runs

@clewis7 clewis7 merged commit 959006e into main Jan 21, 2025
16 checks passed
@clewis7 clewis7 deleted the fix-screenshot branch January 21, 2025 14: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.

2 participants