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

AnimationTree(AnimationNodeStateMachine) error reported in the editor #78052

Closed
magian1127 opened this issue Jun 9, 2023 · 3 comments · Fixed by #79588
Closed

AnimationTree(AnimationNodeStateMachine) error reported in the editor #78052

magian1127 opened this issue Jun 9, 2023 · 3 comments · Fixed by #79588

Comments

@magian1127
Copy link
Contributor

Godot version

4.1b

System information

win11 Vulkan API 1.3.236

Issue description

Selecting AnimationTree will result in error message
This function in this node can only be accessed from either the main thread or a thread group. Use call_deferred() instead.

Steps to reproduce

AnimationTree -> ROOT -> AnimationNodeStateMachine ->Add Animation.
There are several animations, clicking AnimationTree will result in several errors

Minimal reproduction project

test.zip

@AThousandShips
Copy link
Member

AThousandShips commented Jun 9, 2023

This is due to recent threading improvements and seems some of it affects editor functionality

@magian1127
Copy link
Contributor Author

4.1 B2
This function in this node (/root/@EditorNode@16972/@Control@6/@Panel@7/@VBoxContainer@15/@HSplitContainer@18/@HSplitContainer@26/@HSplitContainer@34/@VBoxContainer@35/@VSplitContainer@37/@PanelContainer@6807/@VBoxContainer@6808/@AnimationTreeEditor@12377) can only be accessed from either the main thread or a thread group. Use call_deferred() instead.

@magian1127
Copy link
Contributor Author

Error by animation_tree_editor_plugin.cpp 224 row

Vector<String> AnimationTreeEditor::get_animation_list() {
	if (!singleton->is_visible()) { // this
		return Vector<String>();
	}

'is_visible()' triggered ERR_READ_THREAD_GUARD_V(false)

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.

4 participants