-
-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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
WebGPU backend incorrectly assumes WGPUTextureView pointers will never be reused #7765
Comments
Hello, Do you have a suggested solution as to how to decently solve this? |
I'm not sure WebGPU exposes any kind of truly unique identifier for resources. The only decent solution I can think of is to remove the cache, use WGPUBindGroup as ImTextureID, and provide a helper function for the user to create bind groups from texture views. It would unfortunately require the user to manually manage those bind groups, which kind of feels like it goes against the spirit of IMGUI. |
I don't really know/understand WebGPU yet but I would assume we could call So we remove
|
That sounds like a reasonable solution. I'm new to WebGPU as well, so that didn't even occur to me. It might be worth supporting multiple bind groups for "chunks", since Edit: Actually, I think such a solution would require the proposed |
Let's start with a single bind group and we can easily change it to multiple of them based on runtime limits. |
I don't have test code for loading a texture in the WebGPU example so one would be helpful but otherwise I'll work toward one eventually. |
Hello @dkaste, could you share your workaround solution? I'm facing the same issue and a temporary work around would help. Thanks! Edit: never mind, was able to figure it out. Just called |
As mentioned, I would likely be able to tackle the issue if I had an example of loading textures with WebGPU, e.g a WebGPU version of the tutorial here: https://github.com/ocornut/imgui/wiki/Image-Loading-and-Displaying-Examples |
I am also interested by this issue, but I'm not sure how to contribute to the wiki to add the WebGPU example for loading and displaying an image. I looked around for wiki contributing guidelines but I couldn't find them. Should I just clone the wiki repo and push my modifications to a branch ? |
I am sorry had to disable wiki editing because it prevented google indexing :( see https://github.com/orgs/community/discussions/133123 (please upvote or comment while there). If you submit the contents as a pr in a dummy markdown file i will add it in the meanwhile. |
Thank you for the quick answer. Here is the dummy PR: #8025 |
Version/Branch of Dear ImGui:
Version 1.90.6, Branch: docking (through cimgui)
Back-ends:
imgui_impl_glfw.cpp + imgui_impl_wgpu.cpp
Compiler, OS:
Windows 10 + Zig (MSVC)
Full config/build information:
No response
Details:
My Issue/Question:
The WebGPU backend keeps a cache of
WGPUBindGroup
s that internally referenceWGPUTextureView
s which are simply typedef'd opaque pointers. The backend uses a hash of the texture view pointer as the key for the cache. If a texture view is destroyed and another is created with the same pointer, the old, stale bind group will be used.I ran into this when rapidly creating/destroying textures to resize a custom 3D viewport embedded in IMGUI.
My current workaround is to manually clear the internal bind group cache whenever changing textures used in IMGUI. It works, but it's far from ideal.
Screenshots/Video:
No response
Minimal, Complete and Verifiable Example code:
No response
The text was updated successfully, but these errors were encountered: