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

Objects menu shows errors when going through pages quickly. #216

Open
GrimPixel opened this issue Dec 25, 2022 · 5 comments
Open

Objects menu shows errors when going through pages quickly. #216

GrimPixel opened this issue Dec 25, 2022 · 5 comments
Labels
bug Something isn't working ui Related to the UI upstream Issue comes from upstream

Comments

@GrimPixel
Copy link

GrimPixel commented Dec 25, 2022

Describe the bug
The game gets frozen when there are a lot of objects. Some error messages may appear.

To Reproduce
Steps to reproduce the behavior:

  1. Download https://codeberg.org/GrimPixel/Tabletop-Club-Assets and place them in the assets folder
  2. Stay at the home screen for about twenty seconds
  3. Browse “edu: the flags” in “Object”
  4. Flip the pages quickly
  5. See error

Expected behavior
Nothing happens.

Screenshots
ksnip_20221225-022254
ksnip_20221225-022912

Environment
Arch Linux

Version
v0.1.0, ...

@drwhut drwhut added the bug Something isn't working label Dec 30, 2022
@drwhut drwhut closed this as completed in dcb084d Mar 9, 2023
@drwhut
Copy link
Owner

drwhut commented Mar 9, 2023

I managed to fix two of the issues that I could replicate (the !data.is_inside_tree() errors, and the freezing without any errors) - let me know if you still have any issues related to this.

@GrimPixel
Copy link
Author

It still occurs sometimes. The frequency is much lower than before.

@drwhut
Copy link
Owner

drwhut commented Mar 11, 2023

Good to know - what kind of errors are you seeing now? Errors that are shown in the chat box, or the freezing, or something new?

@GrimPixel
Copy link
Author

It was what the second image shows.

@drwhut
Copy link
Owner

drwhut commented Mar 11, 2023

Hmm... the p_elem->_root != this error never occured for me during testing. I've had a look at Godot's issues, and this particular error has cropped up a bunch of times over the years in multiple different areas, but there's still an open issue here: godotengine/godot#62970, and a closed one that looks very similar here: godotengine/godot#8691

There's one particular situation that keeps showing up in these issues, when add_child is used inside of a thread - as far as I'm aware the code never adds a node to the scene tree inside a thread (instead, I've always made sure that the nodes enter an array with a mutex, then add the node to the scene via the _process() function in the main thread), and according to the docs making a node structure outside of the scene tree in another thread is okay.

So I'm not 100% sure what's going on to cause that error - I'll try to replicate it myself, but I may need help testing.

@drwhut drwhut reopened this Mar 11, 2023
@drwhut drwhut changed the title Game is sometimes frozen when loading many objects Objects menu shows errors when going through pages quickly. Mar 11, 2023
@drwhut drwhut added ui Related to the UI upstream Issue comes from upstream labels Mar 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ui Related to the UI upstream Issue comes from upstream
Projects
None yet
Development

No branches or pull requests

2 participants