Skip to content

Support KHR_texture_basisu extension#392

Open
aW4KeNiNG wants to merge 5 commits intogltf-rs:mainfrom
aW4KeNiNG:KHR_texture_basisu
Open

Support KHR_texture_basisu extension#392
aW4KeNiNG wants to merge 5 commits intogltf-rs:mainfrom
aW4KeNiNG:KHR_texture_basisu

Conversation

@aW4KeNiNG
Copy link

Initial support for the KHR_texture_basisu extension.
Extension information: https://github.com/KhronosGroup/glTF/blob/main/extensions/2.0/Khronos/KHR_texture_basisu/README.md

@mcobzarenco
Copy link

mcobzarenco commented Jan 2, 2025

@alteous Hi, sorry to bother you, is there any way we could get this merged please? I am happy to adopt it if @aW4KeNiNG is not available and make any changes to the PR. I am trying to add support for GLTFs with compressed textures to Bevy.

# Conflicts:
#	gltf-json/src/extensions/texture.rs
#	gltf-json/src/image.rs
#	gltf-json/src/texture.rs
#	src/texture.rs
@alteous
Copy link
Member

alteous commented Jan 13, 2025

I have rebased the changes here: alteous@70312ff

Do these look good to you? I can update the PR if so and get them merged in.

@mcobzarenco
Copy link

mcobzarenco commented May 11, 2025

Yes, thank you, this branch works fine for me, sorry for belated reply. Have been using it in a fork for a while. I also rebased the changes on latest main #456 hoping it's easier to have something you can just merge 🙏

mholiv added a commit to mholiv/gltf that referenced this pull request Feb 16, 2026
Adds a KHR_texture_basisu feature flag that lets textures reference Basis Universal (KTX2) compressed images via the extension's `source` field. Follows the same pattern as EXT_texture_webp: a TextureBasisu
struct, a check in primary_source(), and "image/ktx2" in the valid MIME types list.

Based on gltf-rs#392 by @aW4KeNiNG. The original PR had drifted pretty far behind main over time, so I rebased the basisu bits onto current main.
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 this pull request may close these issues.

3 participants