-
Notifications
You must be signed in to change notification settings - Fork 846
Fix "ToSubgraph" with multiple outputs and allow passthrough #2074
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
Conversation
…ckboard consistent with how nodes behave in searching for properties/keywords
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, "convert to subgraph" no longer detecting recursions is a regression, now that I checked. This should be fixed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested again to make sure I'm not wasting your time, and it does detect recursion with convert to subgraph. Must have made a mistake. Reapproving
…ogies/Graphics into sg/fix-mult-output-convert
Had to fix how we create the subgraph when using the contextual "ToSubgraph" option in the MaterialGraphEditWindow. Depends on fixes to property copying from #1932
for https://fogbugz.unity3d.com/f/cases/1256975/
redid repro steps and could not get bug to occur