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

[BUG] "graph.onGraphChanges += ProcessGraphWhenChanged;" accumulates infinitely #30

Closed
danholbertunity opened this issue Jun 25, 2021 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@danholbertunity
Copy link

What happened?
You can get into a situation where the graph runs an arbitrary number of times upon changes (instead of just one time).

To Reproduce
Open a Mixutre graph.
Close the graph editor.
Open the same graph again.
Attach a debugger and put a breakpoint in MixtureGraphProcessor.ProcessNodeList
Make a change to your graph that would trigger it to run again.
Notice that your breakpoint is hit twice.
Close the graph editor and repeat the process.
Notice that your breakpoint is hit three times.
And so on.

Setup:
Windows
"version": "0.3.0",
"unity": "2020.3",

@danholbertunity danholbertunity added the bug Something isn't working label Jun 25, 2021
@alelievr
Copy link
Owner

Thanks for the bug report, the fix seems straightforward, I'll fix it soon

@alelievr
Copy link
Owner

Fixed in b5238be

@danholbertunity
Copy link
Author

danholbertunity commented Jun 27, 2021 via email

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

2 participants