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

WGPU 0.18.0 Not binding Texture Arrays in GL on windows. #4665

Open
genusistimelord opened this issue Nov 10, 2023 · 5 comments
Open

WGPU 0.18.0 Not binding Texture Arrays in GL on windows. #4665

genusistimelord opened this issue Nov 10, 2023 · 5 comments
Labels
api: gles Issues with GLES or WebGL type: bug Something isn't working

Comments

@genusistimelord
Copy link
Contributor

Description
In vulkan and DX12 the Textures arrays are bound successfully and They currently only have one layer as well. When attempting to test GL for windows everything but the texture arrays bind successfully. You can find the Demo used to test this here: https://github.com/AscendingCreations/render_demo

Here is the RenderDocs file
glcap.zip

here is a screenshot of it showing the textures not being bound.
image

Expected vs observed behavior
Expected the texture to be bound and used. Also expected the limit of arrays to throw an exception if the texture array got to large to not work with the adapter.

Platform
Windows 11
Nividia Geforce GTX 950
WGPU 0.18.0

@teoxoy
Copy link
Member

teoxoy commented Nov 10, 2023

Does it work if you force it start with 2 layers?

@teoxoy teoxoy added type: bug Something isn't working api: gles Issues with GLES or WebGL labels Nov 10, 2023
@genusistimelord
Copy link
Contributor Author

yes it does work if I force it to start with 2 layers.

@teoxoy
Copy link
Member

teoxoy commented Nov 10, 2023

This is most likely due to

// HACK: detect a cube map; forces cube compatible textures to be cube textures
match (desc.is_cube_compatible(), desc.size.depth_or_array_layers) {
(false, 1) => glow::TEXTURE_2D,
(false, _) => glow::TEXTURE_2D_ARRAY,
(true, 6) => glow::TEXTURE_CUBE_MAP,
(true, _) => glow::TEXTURE_CUBE_MAP_ARRAY,
}

Related or duplicate issue: #4081.

@teoxoy
Copy link
Member

teoxoy commented Nov 10, 2023

This will be addressed once we add textureBindingViewDimension to the texture descriptor.

See WIP Proposal Doc & gpuweb/gpuweb#4266.

@genusistimelord
Copy link
Contributor Author

ahh ok for now setting it to multiple layers will fix the issue till you get this fixed on the backend.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: gles Issues with GLES or WebGL type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants