Skip to content

GLTFLoader: support MSFT DDS texture sources (MSFT_texture_dds)#265

Merged
TheMostDiligent merged 1 commit into
DiligentGraphics:masterfrom
hzqst:RTXPT
Jun 12, 2026
Merged

GLTFLoader: support MSFT DDS texture sources (MSFT_texture_dds)#265
TheMostDiligent merged 1 commit into
DiligentGraphics:masterfrom
hzqst:RTXPT

Conversation

@hzqst

@hzqst hzqst commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds support in the AssetLoader glTF pipeline for selecting DDS image sources declared via the MSFT_texture_dds extension, allowing DDS textures to be loaded when present (and reducing spurious “missing standard image” warnings when DDS is used as the supported alternative).

Changes:

  • Parse MSFT_texture_dds on tinygltf::Texture objects and prefer the extension-provided image source during texture loading.
  • Add warning post-processing to suppress tinygltf “missing standard image” warnings when DDS alternatives are available.
  • Introduce helper utilities for source validation, image availability checks, and warning string cleanup.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread AssetLoader/src/GLTFLoader.cpp Outdated
Comment thread AssetLoader/src/GLTFLoader.cpp Outdated
Comment thread AssetLoader/src/GLTFLoader.cpp Outdated
Comment thread AssetLoader/src/GLTFLoader.cpp Outdated
@TheMostDiligent TheMostDiligent merged commit 9413387 into DiligentGraphics:master Jun 12, 2026
55 of 56 checks passed
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