godotengine
/
godot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

GLTF: Fixed external images getting embedded on import 

Added a map to keep track of external images during import, and used
that map to instance the textures using the resource loader instead of
creating a new texture from scratch
This commit is contained in:
pablitar 2024-07-22 13:51:53 -03:00
parent 2cc5ca43ef
commit 601a9eb4f5
2 changed files with 18 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
modules/gltf/gltf_document.cpp
View File

@ -3106,6 +3106,7 @@ Error GLTFDocument::_parse_images(Ref<GLTFState> p_state, const String &p_base_p
Ref<Texture> texture = ResourceLoader::load(uri);
if (texture.is_valid()) {
p_state->images.push_back(texture->get_data());
p_state->external_images_paths.insert(i, uri);
continue;
} else if (mimetype == "image/png" || mimetype == "image/jpeg") {
// Fallback to loading as byte array.
@ -3234,17 +3235,25 @@ Error GLTFDocument::_parse_textures(Ref<GLTFState> p_state) {
}
p_state->textures.push_back(t);
// Create and cache the texture used in the engine
Ref<ImageTexture> imgTex;
imgTex.instance();
imgTex->create_from_image(p_state->images[t->get_src_image()]);
Ref<Texture> tex;
// Set texture filter and repeat based on sampler settings
// Create and cache the texture used in the engine
if (p_state->external_images_paths.has(t->get_src_image())) {
tex = ResourceLoader::load(p_state->external_images_paths[t->get_src_image()]);
} else {
Ref<ImageTexture> img_tex;
img_tex.instance();
img_tex->create_from_image(p_state->images[t->get_src_image()]);
// Set texture filter and repeat based on sampler settings. Only supported for embedded textures
const Ref<GLTFTextureSampler> sampler = _get_sampler_for_texture(p_state, i);
Texture::Flags flags = sampler->get_texture_flags();
imgTex->set_flags(flags);
img_tex->set_flags(flags);
p_state->texture_cache.insert(i, imgTex);
tex = img_tex;
}
p_state->texture_cache.insert(i, tex);
}
return OK;

1
modules/gltf/gltf_state.h
View File

@ -78,6 +78,7 @@ class GLTFState : public Resource {
Vector<Ref<GLTFTextureSampler>> texture_samplers;
Ref<GLTFTextureSampler> default_texture_sampler;
Vector<Ref<Image>> images;
Map<GLTFImageIndex, String> external_images_paths;
Map<GLTFTextureIndex, Ref<Texture>> texture_cache;
Vector<String> extensions_used;
Vector<String> extensions_required;