Skip to content

Conversation

@julienf-unity
Copy link
Contributor

Purpose of this PR

Backport 2020.2 of #3498
Fix for https://fogbugz.unity3d.com/f/cases/1300320/


Testing status

Tested locally


Comments to reviewers

HUP bug

@julienf-unity julienf-unity requested review from a team and PaulDemeulenaere February 15, 2021 10:53
@github-actions github-actions bot added the vfx label Feb 15, 2021
@github-actions
Copy link

It appears that you made a non-draft PR!
Please convert your PR to draft (button on the right side of the page)
and cancel any jobs that started on Yamato.
See the PR template for more information.
Thank you!

@julienf-unity julienf-unity changed the title Some fixes in expression graph compilation [2020.2][VFX] Fix 1301094 - Infinite compilation loop Feb 15, 2021
@julienf-unity julienf-unity changed the title [2020.2][VFX] Fix 1301094 - Infinite compilation loop [2020.2][VFX] Fix 1300320 - Infinite compilation loop Feb 15, 2021
@VitaVFX VitaVFX requested review from VitaVFX and removed request for a team February 15, 2021 11:25
@VitaVFX
Copy link

VitaVFX commented Feb 18, 2021

Hi, here's the Test Doc. Thank you for fixing this nasty bug, although found a few issues that could be looked into

@VitaVFX
Copy link

VitaVFX commented Feb 23, 2021

The issue Vlad found regarding changing settings of disabled blocks recompilation is also reproducible on 2020.2. Could this be fixed in 2020.2 too? Test Doc

Copy link

@VitaVFX VitaVFX left a comment

Choose a reason for hiding this comment

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

Looks good, haven't find any other issues!

@julienf-unity julienf-unity merged commit 870c12e into 10.x.x/release Feb 24, 2021
@julienf-unity julienf-unity deleted the 10.x.x/vfx/fix/1300320 branch February 24, 2021 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants