volumeRenderingTexture3D bug?

[greggman]

When I go to: https://webgpu.github.io/webgpu-samples/?sample=volumeRenderingTexture3D

I get

Uncaptured error:
Required size for texture data layout (6998400) exceeds the linear data size (3788907) with offset (0).

• While calling [Queue].WriteTexture([TexelCopyTextureInfo texture: [Texture (unlabeled 180x216x180 px, TextureFormat::R8Unorm)], mipLevel: 0, origin: [Origin3D x:0, y:0, z:0], aspect: TextureAspect::All], (3788907 bytes), [TexelCopyBufferLayout offset:0, bytesPerRow:180, rowsPerImage:216], [Extent3D width:180, height:216, depthOrArrayLayers:180])

[greggman]

Strange, I don't get this when running from localhost on top of tree

[greggman]

Just guessing it's because these files end in .gz and are treated differently by the github pages server vs the dev server.?

[greggman]

Yea, when served from the dev server

content-encoding: gzip
content-length: 3788907
content-type: application/octet-stream

when served from github pages its

content-length: 3788907
content-type: application/gzip

content-encoding so the browser doesn't auto-un-gzip

github pages will auto compress certain files types. We probably shouldn't be using .gz and just include the uncompressed versions and let the server decide whether or not to serve them compressed?

Should I turn off the content-type on the dev server so it matches github pages?

[greggman]

related:

• https://github.com/orgs/community/discussions/21655
• Support precompressed .gz files on GitHub Pages (like nginx’s gzip_static) isaacs/github#611

[greggman]

It's not clear what extension to use .bin files are not auto-compressed by github pages. .glb are .wasm are. There is apparently no official list from github of the extensions that are served compressed.

If we just want the sample to work we can just uncompress them as their .bin files

If we want to download and decompress them ourselves there's unzipit which I wrote. It's minimal, low-dependency, and has a simple interface

I think I'd opt for .bin files.

[kainino0x]

This was working before #501, and we were already using gzip for the non-gpu-compressed data. Was this caused by the switch from .bin-gz to .bin.gz?

[kainino0x]

We can always do a ServiceWorker hack, e.g. https://github.com/jprendes/service-worker-brotli

[beaufortfrancois]

I think we can revert af6432c only then.

[greggman]

This was working before #501, and we were already using gzip for the non-gpu-compressed data. Was this caused by the switch from .bin-gz to .bin.gz?

Testing in another repo .bin-gz isn't working for me on github. Maybe it never worked and we always tested locally?

[greggman]

ah, I see from @beaufortfrancois's comment, it used to be decompressing manually (and I learned about DecompressionStream which I'd never heard of). Yea, they should probably stay .bin-gz (so they'll be more likely to work on any server) and keep the manual decompression?

[beaufortfrancois]

https://webgpu.github.io/webgpu-samples/?sample=volumeRenderingTexture3D now works great! Thank you @greggman and @kainino0x!