Skip to content

Fixes for 5.1 "red nodes of death" #27

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

Merged
merged 5 commits into from
Jan 6, 2023
Merged

Fixes for 5.1 "red nodes of death" #27

merged 5 commits into from
Jan 6, 2023

Conversation

sfjohnston
Copy link
Contributor

On 5.1, the nodes would read into a file red and refreshing nodes did not work.
Changing the loading to "PreDefault" rather than "Default" seems to do the trick.
(Also fixed a braces placement issue which was needed for the linux section.)

@irajsb
Copy link
Owner

irajsb commented Jan 3, 2023

I could not reproduce the bugs. If the missing function was import scenes async it was removed recently because scene importing was fast enough and it didn't needed to be async(for now ).

@irajsb irajsb merged commit 9b0ea5c into irajsb:main Jan 6, 2023
@sfjohnston
Copy link
Contributor Author

The issue in the pull request was not about async scene import.

On MacOS and Linux with UE5.1, all the nodes in a blueprint would be "red" when the editor was opened. Performing a "refresh all nodes" partially fixes the problem, but some connections have the wrong type at that point.
I compared this plugin to another that doesn't have the issue, and the difference was the "PreDefault" vs. "Default" setting. When I switched the load setting to "PreDefault" the issue went away.

If you're seeing my latest push with three additional changes, you will see that I've made another bug-fix, and a change for clarity.

Bugfix: Instead of using FColor, which maps from sRGB to linear color space, I'm returning the base color created with FLinearColor. This passes the correct colors and no longer needs the *100.

Clarity Change: I was confused that "GetAllMeshes" behaves differently in AiScene and AiNode. It makes sense now, but I've changed the name to "GetNodeMeshes" in AiNode for clarity and symmetry with other calls like "GetNodeName" and "GetNodeTransform." (I'm tempted to similarly change GetParentNode to GetNodeParent and GetChildNodes to GetNodeChildren).

The third change is updating the Assimp submodule to the latest version, which has some bug fixes to quiet down compiler warnings on macOS.

Thank you for accepting these changes!

@sfjohnston
Copy link
Contributor Author

sfjohnston commented Jan 6, 2023 via email

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

Successfully merging this pull request may close these issues.

2 participants