Skip to content

Suppress warnings for optional texture lookups - #161

Merged
anirul merged 1 commit into
masterfrom
codex/suppress-optional-texture-lookup-warnings
Apr 19, 2026
Merged

Suppress warnings for optional texture lookups#161
anirul merged 1 commit into
masterfrom
codex/suppress-optional-texture-lookup-warnings

Conversation

@anirul

@anirul anirul commented Apr 19, 2026

Copy link
Copy Markdown
Owner

Summary

  • add quiet texture-name lookup helpers for OpenGL and Vulkan program parsing
  • use the quiet lookup path when probing optional glTF/material texture bindings
  • preserve the existing fallback behavior without emitting warning spam for expected misses

Why

Some render paths intentionally probe optional texture names such as opaque_*, transmissive_*, and material-specific bindings. Those probes were using GetIdFromName(...), which logs a warning on every miss. In scenes that rely on imported material bindings instead of predeclared level textures, this produced a large amount of misleading log noise even though rendering still worked.

Validation

  • rebuilt grpcmmo_client against this Frame change
  • reran the client and checked the generated log.txt
  • confirmed the No id for name ... warning flood is gone
  • remaining warnings are only the unrelated Vulkan validation-layer environment warnings

@anirul
anirul merged commit c8464d6 into master Apr 19, 2026
1 check passed
@anirul
anirul deleted the codex/suppress-optional-texture-lookup-warnings branch April 19, 2026 12:35
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.

1 participant