-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[gui] GGUI Image IO (well, it's actually just O...) #3333
Conversation
✔️ Deploy Preview for jovial-fermat-aa59dc canceled. 🔨 Explore the source changes: 3f81d1b 🔍 Inspect the deploy log: https://app.netlify.com/sites/jovial-fermat-aa59dc/deploys/617e688a0a9d23000804a3a6 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work!
9db029c
to
3f81d1b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
help me merge it? :D |
3333! |
This PR adds image IO for GGUI. The implementation is somewhat based on this.
Notice that this PR alone doesn't enable headlessly running GGUI: it only allows you to dump an image file while the window is also running.
In the next PR, I'll add in headless support. And then we can start adding tests for GGUI.
Not working on MoltenVK on macOS... will try to fix.
And finally, this PR should be merged after #3330, otherwise GGUI won't work at all.