Skip to content

update screenshot #700

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

Merged
merged 3 commits into from
Jan 21, 2025
Merged

update screenshot #700

merged 3 commits into from
Jan 21, 2025

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
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy