Skip to content

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

Merged
merged 25 commits into from
Dec 15, 2020

Conversation

elizabeth-legros
Copy link
Contributor

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

@elizabeth-legros elizabeth-legros marked this pull request as ready for review October 19, 2020 16:41
@elizabeth-legros elizabeth-legros requested a review from a team as a code owner October 19, 2020 16:41
@elizabeth-legros elizabeth-legros requested a review from a team October 19, 2020 16:41
@GrantLamb-Unity GrantLamb-Unity requested review from GrantLamb-Unity and a user and removed request for a team and GrantLamb-Unity October 20, 2020 18:37
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ran the following mindmap on subgraphs
Subgraph smoke test
Only found 1 bug and that bug also reproduced on Master.
Approving.

@ghost ghost self-requested a review October 21, 2020 20:56
Copy link

@ghost ghost left a 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.

Copy link

@ghost ghost left a 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

@cdxntchou cdxntchou merged commit 4d843ce into master Dec 15, 2020
@cdxntchou cdxntchou deleted the sg/fix-mult-output-convert branch December 15, 2020 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants