Skip to content

Conversation

PaulDemeulenaere
Copy link
Contributor

Purpose of this PR

Backport of #6475


Testing status

Yamato ⏳


Comments to reviewers

N/A

* Fix TransformNode usage in VFX

Missing World To Object matrix requirement

* *Update changelog

Only in VFX, I don't see other cases where this fix is applicable, maybe DOTS ?

* Better filtering of needed transform

Fix issue #6475 (comment)
Fix issue #6475 (comment)
# Conflicts:
#	com.unity.visualeffectgraph/CHANGELOG.md
@Unity-Technologies Unity-Technologies deleted a comment from github-actions bot Jan 27, 2022
@PaulDemeulenaere PaulDemeulenaere marked this pull request as ready for review January 27, 2022 18:54
@PaulDemeulenaere PaulDemeulenaere requested a review from a team as a code owner January 27, 2022 18:54
@PaulDemeulenaere PaulDemeulenaere requested review from esmelusina and removed request for cdxntchou January 27, 2022 18:56
Copy link
Contributor

@esmelusina esmelusina left a comment

Choose a reason for hiding this comment

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

LGTM

@PaulDemeulenaere
Copy link
Contributor Author

Yamato is green, I double checked the change & resolution of conflicts in changelog, merging now. Thanks!

@PaulDemeulenaere PaulDemeulenaere merged commit e66e349 into 2022.1/staging Jan 28, 2022
@PaulDemeulenaere PaulDemeulenaere deleted the 2022.1/vfx/fix/1363279-invalid-tangent-space-using-sg branch January 28, 2022 08:01
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.

2 participants