Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Moving a GLTF separate (.gltf) file with external textures results in it failing to load properly, and error spam #85177

Closed
Jordyfel opened this issue Nov 21, 2023 · 0 comments · Fixed by #94506
Milestone

Comments

@Jordyfel
Copy link
Contributor

Jordyfel commented Nov 21, 2023

Godot version

4.2-beta6

System information

Windows

Issue description

Moving the file seems to completely break gltf separate. Moving the external texture itself doesn't affect anything.

Edit: The reason is that moving the .gltf and its dependencies visible in the editor leaves out the .bin file. It's existence is not obvious and the generated errors could be more helpful.

The generated errors:

  Can't open file from path 'res://untitled.bin'.
  modules/gltf/gltf_document.cpp:1323 - Condition "(int)(offset + buffer_end) > buffer.size()" is true. Returning: ERR_PARSE_ERROR
  modules/gltf/gltf_document.cpp:1323 - Condition "(int)(offset + buffer_end) > buffer.size()" is true. Returning: ERR_PARSE_ERROR
  modules/gltf/gltf_document.cpp:2868 - Condition "vertex_num <= 0" is true. Returning: ERR_INVALID_DECLARATION
  modules/gltf/gltf_document.cpp:7251 - Condition "err != OK" is true. Returning: ERR_PARSE_ERROR
  modules/gltf/gltf_document.cpp:6895 - Condition "err != OK" is true. Returning: err
  modules/gltf/gltf_document.cpp:7426 - Condition "err != OK" is true. Returning: err
  Error importing 'res://untitled.gltf'.

Steps to reproduce

  1. Download the minimum reproduction project
  2. Open the scene
  3. Move untitled.gltf into the additional dir, with or without the texture
  4. Observe undesired behavior

Minimal reproduction project

test_proj.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants