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

Importing glTF with nested structure gives "inconsistent owner" warning #88125

Closed
stevenctl opened this issue Feb 9, 2024 · 3 comments · Fixed by #88497
Closed

Importing glTF with nested structure gives "inconsistent owner" warning #88125

stevenctl opened this issue Feb 9, 2024 · 3 comments · Fixed by #88497

Comments

@stevenctl
Copy link

Tested versions

  • Reproducible in 4.3-dev3
  • Not reproducible in 4.2 stable

System information

Linux 6.1 - Godot 4.3-dev3 - OpenGL Compat backend

Issue description

When importing a gLTF file with some nested hierarchy, each child of the scenes root (and nested children) emit the following warning:

  scene/main/node.cpp:1432 - Adding 'Rope' as child to 'Mast' will make owner 'pirateship' inconsistent. Consider unsetting the owner beforehand.
  scene/main/node.cpp:1432 - Adding 'Sail' as child to 'Mast' will make owner 'pirateship' inconsistent. Consider unsetting the owner beforehand.
  scene/main/node.cpp:1432 - Adding 'RearFlag' as child to 'RearFlagPole' will make owner 'pirateship' inconsistent. Consider unsetting the owner beforehand.
  scene/main/node.cpp:1432 - Adding 'Door' as child to 'Ship' will make owner 'pirateship' inconsistent. Consider unsetting the owner beforehand.
  scene/main/node.cpp:1432 - Adding 'Mast' as child to 'Ship' will make owner 'pirateship' inconsistent. Consider unsetting the owner beforehand.
  scene/main/node.cpp:1432 - Adding 'RearFlagPole' as child to 'Ship' will make owner 'pirateship' inconsistent. Consider unsetting the owner beforehand.
  scene/main/node.cpp:1432 - Adding 'Wheel' as child to 'Ship' will make owner 'pirateship' inconsistent. Consider unsetting the owner beforehand.

This doesn't seem to cause any real issues and ownership ends up being fine.

Steps to reproduce

Simply import the glb. You don't need to instance it in a scene or do anything else.

Minimal reproduction project (MRP)

repro.zip

@Rindbee
Copy link
Contributor

Rindbee commented Feb 13, 2024

Related to #78997, which did not take into account the case where the sub-scene root node was replaced_by.

@akien-mga akien-mga added this to the 4.3 milestone Feb 15, 2024
@akien-mga akien-mga changed the title Importing gLTF with nested structure gives "inconsistent owner" warning Importing glTF with nested structure gives "inconsistent owner" warning Feb 15, 2024
@akien-mga
Copy link
Member

CC @Sauermann

@dandeliondino
Copy link

I'm getting the same error when importing .blend files in 4.2.2 RC1. Error does not occur on 4.2.1-stable with the same files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Immediate Blocker
Development

Successfully merging a pull request may close this issue.

6 participants