Skip to content
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

Add glfwCreateWindowWGPUSurface for native WebGPU #2333

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

eliemichel
Copy link

As noticed in #2243 it seems logical that GLFW provides a glfwCreateWindowWGPUSurface for WebGPU similar to the glfwCreateWindowSurface it has for Vulkan, as this is the only part of the process of creating a "hello triangle" in native WebGPU that requires OS-specific code (more details).

I am not sure about all the details of this PR, I know the original glfw3webgpu repo works well but maybe I did not integrate it as well as I could in this project's CMakeLists!

@eliemichel eliemichel changed the title Add glfwCreateWindowWGPUSurface Add glfwCreateWindowWGPUSurface for native WebGPU May 3, 2023
@Kangz
Copy link
Contributor

Kangz commented May 4, 2023

Use of the WebGPU header LGTM FWIW and I think wgpuInstanceCreateSurface will likely never change. @cwfitzgerald does that match your understanding as well?

@cwfitzgerald
Copy link

It does. It shouldn't change meaningfully, the API already done completely through extension structs.

@elmindreda elmindreda added this to Needs Investigation in Queue Feb 14, 2024
@eliemichel
Copy link
Author

Hey @elmindreda I see you've added this to your "Needs Investigation" list, is there any way I can help?

FWIW I've been actively using this for the past year, and it is a part of the LearnWebGPU guide without trouble, so it's not just me!

Copy link

@object71 object71 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@eliemichel
Copy link
Author

WIP support for the new runtime platform switch (X11/Wayland): eliemichel/glfw3webgpu#8 Will be ported to this PR once tested enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Queue
Needs Investigation
Development

Successfully merging this pull request may close these issues.

None yet

4 participants