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

Scene export duplication error #96709

Closed
DragonMaster14545 opened this issue Sep 8, 2024 · 3 comments
Closed

Scene export duplication error #96709

DragonMaster14545 opened this issue Sep 8, 2024 · 3 comments

Comments

@DragonMaster14545
Copy link

DragonMaster14545 commented Sep 8, 2024

Tested versions

reproducable in v4.3.stable.steam [77dcf97]

System information

Godot v4.3.stable (77dcf97) - Windows 10.0.22631 - Vulkan (Mobile) - dedicated NVIDIA GeForce RTX 4070 Laptop GPU (NVIDIA; 31.0.15.3820) - Intel(R) Core(TM) i9-14900HX (32 Threads)

Issue description

Hi, so I experienced a similar issue a few times already, but I never was motivated to get a MRP up and running since I don't have any idea why this happens. So, this has to do something with creating a new node on runtime, adding it to the scene tree, propperly setting the owner and then going to the remote tree and saying export as scene on the root node or the node directly below it. This will give the following errors:

  <C++-Quelle>   scene/main/node.cpp:1688 @ get_child()
  <C++-Fehler>   Parameter "copy_child" is null.
  <C++-Quelle>   scene/main/node.cpp:2926 @ _duplicate_properties()

The export itsself is fine, but the through code added node is gone
This is verry anoying, because I often want to open my scene in the editor for debugging, and having a lot of stuff missing kinda defeats that purpose.
Thanks in advance

Steps to reproduce

Open the MRP, just click play, then go to the remote scene tree, right click either the root node or the Node3D node on the top, and the click export as scene and click ok

Minimal reproduction project (MRP)

MRP.zip

@AThousandShips
Copy link
Member

Thank you for reporting, consolidating in:

See there for more details, if you think something was missed about this and it's not the same issue, please comment here and it can be reopened

@AThousandShips AThousandShips closed this as not planned Won't fix, can't repro, duplicate, stale Sep 9, 2024
@DragonMaster14545
Copy link
Author

@AThousandShips I just tested, my issue is not only with the model, I just swapped out the model loading with a simple Node3D, exact same error
MRP new.zip

@AThousandShips
Copy link
Member

It's the same issue with the same origin, please test the PR:

And add any new information to the other report

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

No branches or pull requests

2 participants