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

chore: update unity project version to unity2022.3.6f1 #5529

Merged
merged 35 commits into from
Aug 29, 2023

Conversation

pravusjif
Copy link
Member

@pravusjif pravusjif commented Jul 26, 2023

  • Updated whole project from Unity 2021.3.22f1 LTS to Unity 2022.3.6f1 LTS
  • Updated UberPost shader based on URP 14.0.8. Re-applying our alpha value fix
  • Updated CharacterPreview prefab camera background color from white to black to improve avatars blending with the passport screen background
  • As per 2022.3.6f1 the annoying "FMOD errors" in the WebGL build should've gone away
  • Updated CI images used
  • Tried to replace the RenderTargetHandle usages (marked as obsolete since URP 13) with the new RTHandle class but couldn't get it to work. Since in URP 14.0.8 (the one used with Unity 2022.3) the old "obsolete" class still works, I'm leaving that implementation as it was. I created an informative draft PR to tackle that in the future, since probably that change will be mandatory to update the URP version further.

TESTING

This build should be tested as throroughly as a Release Candidate, and also adding:

  • Creating a new avatar from scratch as a guest
  • Creating a new avatar from scratch as a non-guest
  • Checking if the popular scenes work/look the same way as in production (ICE Poker scene, Wonder Mine, GolfCraft, any other scene that contains video playing)

TEST LINKS

@pravusjif pravusjif self-assigned this Jul 26, 2023
@pravusjif pravusjif changed the title chore: unity2022.3.4 chore: unity2022.3.6f1 Aug 4, 2023
@pravusjif
Copy link
Member Author

pravusjif commented Aug 23, 2023

Pending:

  • At some point when updating with dev there's some problem in kernel that breaks the usage of the new build...
  • The CI has some weird error as well...

@pravusjif pravusjif changed the title chore: unity2022.3.6f1 chore: update unity project version to unity2022.3.6f1 Aug 23, 2023
… if CI problem comes from cached libraries

Signed-off-by: Pravus <[email protected]>
…l to see if CI problem comes from cached libraries"

This reverts commit 8e8a23b.
@pravusjif
Copy link
Member Author

the CI error was due to using cached libraries for Unity 2021.3.22 instead of the new version

@pravusjif pravusjif marked this pull request as ready for review August 24, 2023 10:21
@pravusjif pravusjif requested a review from a team as a code owner August 24, 2023 10:21
@anicalbano anicalbano self-requested a review August 24, 2023 16:05
Copy link
Contributor

@anicalbano anicalbano left a comment

Choose a reason for hiding this comment

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

✅ PR reviewed by QA. No issues where found on our checks.

Copy link
Contributor

@Kinerius Kinerius left a comment

Choose a reason for hiding this comment

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

@pravusjif
Copy link
Member Author

I'll update with dev latest changes just before merging to main.

Copy link
Contributor

@sandrade-dcl sandrade-dcl left a comment

Choose a reason for hiding this comment

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

LGTM!

@pravusjif pravusjif added the do not merge This PR should not be merged because is experimental or for testing purposes. label Aug 28, 2023
@pravusjif pravusjif merged commit e0095ef into dev Aug 29, 2023
@pravusjif pravusjif deleted the test/perf-profiling-unity2022.3.4f1 branch August 29, 2023 10:22
pravusjif added a commit that referenced this pull request Aug 30, 2023
pravusjif added a commit that referenced this pull request Aug 30, 2023
Revert "chore: update unity project version to unity2022.3.6f1 (#5529)"

This reverts commit e0095ef.
@pravusjif pravusjif restored the test/perf-profiling-unity2022.3.4f1 branch August 30, 2023 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge This PR should not be merged because is experimental or for testing purposes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants