Improve workaround for wgpu's gles texture type inference about bevy_ecs_tilemap HOT 4 OPEN

bevy_ecs_tilemap/src/render/texture_array_cache.rs

Lines 151 to 152 in 3171dcb

	// Fixes weird cubemap bug.
	let count = if *count == 6 { count + 1 } else { *count };

The current workaround is incomplete, count should not be 1 or divisible by 6.

See gfx-rs/wgpu#3124 (comment) for more context.

I'm not sure why count shouldn't be 1 or divisible by 6? It also looks like wgpu hasn't implemented a fix yet.

from bevy_ecs_tilemap.

I have been meaning to test this more with opengl on the desktop, but I think that on webgl2 our users aren't really affected by this because they need to use the atlas feature to avoid another bug anyway.

I attempted this earlier, but I think there are some other criteria that had me scratching my head at the time:
https://github.com/gfx-rs/wgpu/blob/9dc834a0ac694d576a584a89f72733a0bb2f4485/wgpu-hal/src/lib.rs#L884

from bevy_ecs_tilemap.

I'm not sure why count shouldn't be 1 or divisible by 6?

Since we might use GL_TEXTURE_2D, GL_TEXTURE_CUBE_MAP or GL_TEXTURE_CUBE_MAP_ARRAY based on this heuristic instead of the intended GL_TEXTURE_2D_ARRAY.

from bevy_ecs_tilemap.

I think that the other bug I alluded to gfx-rs/wgpu#3197 has been fixed. It doesn't seem totally necessary to use the atlas feature now, so we should probably take a look at this again.

from bevy_ecs_tilemap.