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

WASM Support for texture arrays #283

Closed
aleokdev opened this issue Sep 23, 2022 · 6 comments · Fixed by #557
Closed

WASM Support for texture arrays #283

aleokdev opened this issue Sep 23, 2022 · 6 comments · Fixed by #557

Comments

@aleokdev
Copy link

The crate doesn't seem to work with WASM, as it doesn't render anything. Tried the LDTK example and got this:
Screenshot_20220923_230924

@StarArawn
Copy link
Owner

You need to use the atlas feature on WASM. There is a wgpu bug with array textures.

@aleokdev
Copy link
Author

Works perfectly, thanks!

@bzm3r
Copy link
Collaborator

bzm3r commented Sep 25, 2022

@StarArawn Can you link to the relevant WGPU issue here? With that, we can close this issue, I think.

@bzm3r bzm3r changed the title WASM Support WASM Support for texture arrays Sep 25, 2022
@StarArawn
Copy link
Owner

@StarArawn Can you link to the relevant WGPU issue here? With that, we can close this issue, I think.

No issue that I know of. I can probably create one though. I need to collect more info though.

@musjj
Copy link

musjj commented Jul 28, 2024

Are there any downsides/overhead to the atlas feature, or is it safe to just activate on all target archs?

@StarArawn
Copy link
Owner

Are there any downsides/overhead to the atlas feature, or is it safe to just activate on all target archs?

There are more rendering artifacts because atlases inherently bleed neighboring pixels. Texture arrays are generally better and are supported by webGPU if that is your target.

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 a pull request may close this issue.

4 participants