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

Background subsystem throws an error when it tries to call transition_finished after background node isnt in tree anymore. #2121

Open
catpieyum opened this issue Mar 1, 2024 · 2 comments
Labels
Bug 🐞 Something isn't working

Comments

@catpieyum
Copy link

The problem

When using the background event with a fade time of lets say 3 seconds, while transitioning, we move onto the next background event or end the timeline, an error pops up saying "Error calling from signal 'transition_finished' to callable: 'Node(subsystem_backgrounds.gd)::_on_transition_finished': Cannot convert argument 1 from Object to Object."

To Reproduce
Steps to reproduce the behavior:

  1. Add background event to timeline with fade time of 0
  2. Add a text event
  3. Add background event to timeline with fade time of 3 without any image

System (please complete the following information):

  • OS: Windows
  • Godot Version: 4.2.1.stable
  • Dialogic Version: 2.0.a13

Possible fixes
Disconnecting transition_finished before freeing the node should probably fix it. Or checking if node still exists when calling transition_finished.

@CakeVR CakeVR added the Bug 🐞 Something isn't working label Mar 8, 2024
@Jowan-Spooner
Copy link
Collaborator

Hey @catpieyum it's been a while, but I'm trying to recreate this, and I'm not getting the error. This is the timeline I'm testing:

[background arg="res://test-project/Backgrounds/Bedroom 27.png" fade="0.0"]
[wait time="1.0"]
[background fade="3.0"]
[wait time="1.0"]
[background arg="res://test-project/Backgrounds/Bedroom 27.png" fade="0.0"]
[wait time="1.0"]
[background fade="3.0"]
[wait time="1.0"]

Am I missing something? Can you share a video of this happening (can be on discord). Or the timeline snippet that does this? Thanks!

@catpieyum
Copy link
Author

I will take a video soon and send it here or in Discord 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug 🐞 Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants