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

FlowcharWindow causes GUI Layout error in the Unity 2021 LTS (Severe BUG!) #1052

Closed
breadnone opened this issue Apr 22, 2022 · 6 comments
Closed
Labels
bug Doing something it's not meant to, or not doing something it's meant to.
Milestone

Comments

@breadnone
Copy link
Contributor

Describe the bug
GUI Layout error will be thrown, every time there is an action in the Flowchart graph in Unity 2021 LTS and will cause StackOverflow exception.

To Reproduce
Install Fungus in Unity 2021 LTS, create a flowchart, then Open Flowchart Window. The error will be shown

Additional context
It's a very sever bug, almost makes Fungus unusable in projects!

@breadnone breadnone added the bug Doing something it's not meant to, or not doing something it's meant to. label Apr 22, 2022
@oldbloodgames
Copy link

Just seconding the issue. This bug essentially makes using Fungus in Unity 2021 impossible with the perpetual crashing from the StackOverflow exception.

@DrXand
Copy link

DrXand commented Apr 22, 2022

Same problem on any version of Unity 2021, LTS or not. Because of this, I can't migrate my project to the new version of the engine.

@addie-lombardo
Copy link

addie-lombardo commented Apr 23, 2022

Was about to finally leave an issue for this and glad to see others have been seeing it too and it's not just me. I did do a quick run through the script and all the Begin/End methods are being called properly (which is what the error warns you about) so I'm at a little bit of a loss as to what changed in the change from 2020 to 2021. I'll dive deeper on this because it needs to be fixed, I'm sick of Unity regularly crashing!

Edit: I narrowed it down to something is happening in the DrawOverlay method of FlowchartWindow which is causing the error to be thrown. I've tried commenting out chunks of it to find what exactly it was but it only fixed the error if I commented out the whole method.

@breadnone
Copy link
Contributor Author

Found a fix for this, will make a pr soon

@stevehalliwell
Copy link
Collaborator

The linked pr seems straightforward enough. But can someone provide detailed repro steps for unity 2021.3.1 as I'm not able to create it other than as a one off when adding a variable to a flowchart via the in flowchartwindow add button. So we can go through the steps to check for other gremlins hiding in there.

@breadnone
Copy link
Contributor Author

breadnone commented May 4, 2022

The linked pr seems straightforward enough. But can someone provide detailed repro steps for unity 2021.3.1 as I'm not able to create it other than as a one off when adding a variable to a flowchart via the in flowchartwindow add button. So we can go through the steps to check for other gremlins hiding in there.

@stevehalliwell Another way to trigger the error is by clicking the flowchart gameObject in the hierarchy then click the Open Flowchart Window. Basically anything that triggers the opening of a flowchart would trigger it.

Another thing that CAN'T trigger the error is when the domain-reloading is done then very quickly(like immediately) open the flowchart window, we can sometimes open the flowchart window without any hiccups/issues

Edit: You should test it in 2021 LTS and above, as they changed some of the old IMGui api in preparation for making the default ui-toolkit for the default Editor api as mentioned here #1051

Edit2 : Docking/Undocking the opened Flowchart window will trigger the error as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Doing something it's not meant to, or not doing something it's meant to.
Projects
None yet
Development

No branches or pull requests

5 participants