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

Internal/2022.3/staging #8043

Merged
merged 5 commits into from
Mar 13, 2024
Merged

Conversation

UnityAljosha
Copy link
Collaborator

Please read the Contributing guide before making a PR.

Checklist for PR maker

  • Have you added a backport label (if needed)? For example, the need-backport-* label. After you backport the PR, the label changes to backported-*.
  • Have you updated the changelog? Each package has a CHANGELOG.md file.
  • Have you updated or added the documentation for your PR? When you add a new feature, change a property name, or change the behavior of a feature, it's best practice to include related documentation changes in the same PR. If you do add documentation, make sure to add the relevant Graphics Docs team member as a reviewer of the PR. If you are not sure which person to add, see the Docs team contacts sheet.
  • Have you added a graphic test for your PR (if needed)? When you add a new feature, or discover a bug that tests don't cover, please add a graphic test.

Purpose of this PR

Why is this PR needed, what hard problem is it solving/fixing?


Testing status

Describe what manual/automated tests were performed for this PR


Comments to reviewers

Notes for the reviewers you have assigned.

ellioman and others added 5 commits February 28, 2024 17:36
**Primary goal was to update UTR, UTF and GTF (Graphics test framework)**
- Backport https://github.cds.internal.unity3d.com/unity/unity/pull/43428.
- Backport https://github.cds.internal.unity3d.com/unity/unity/pull/34689.
- Bump UTR to 1.28.2 and UTF to 1.3.9, in order to catch up with latest fixes and features. SRP Test projects only.
- Bump GTF to 8.3.2. This means updating the client side logic of all `com.unity.testing...` test setup packages as there are breaking changes for android and webgl when switching from GTF 7.x.x.

**I also did some cleaning.**
- Reset `com.unity.testing-graphics-performance` to `14.0.10` (from arbitrary `8.0.0`), as it is always used as a local (in-repo) package and doesn't correspond to any package fetched from artifactory. 14.0.10 allows matching the versionning scheme of the render-pipelines.core dependency, as was discussed [here](https://github.cds.internal.unity3d.com/unity/unity/pull/44608#pullrequestreview-716897).
- Update the `com.unity.test-framework.performance` dependency from `2.0.8-preview` to latest, `3.0.3`. 
- Fix the asmdef references as the graphics perf package was missing a reference on the performance package. I switched from using GUIDs to full names because I believe we should prefer clarity over the only other benefit according [to the docs](https://docs.unity3d.com/Manual/class-AssemblyDefinitionImporter.html#general), which is to not have to update the references when their names change (which basically never happens for these packages - there is close to 0 code churn).

@pauld, added you for all the VFX Graph client side testing library changes.
Added Additional Resources to URP Lighting docs landing page
…eing deallocated on scene changes.

This PR fixes GameObjects allocated by HDRP (CameraCache, HDAdditionalLightData/ComponentSingletons) being deallocated in some cases on scene changes. 
It also slightly simplifies the Probe Camera cache to avoir re-setting the same values every frame. 
This fix was discussed during previous bugfix with @julien.
…ow resolution screen space GI

Fixing artifacts on low resolution SSGI when dynamic resolution values are low. We have added a new setting in HDRP DRS settings to clamp the minimum resolution that SSGI can go with reference to the output resolution.
The denoiser does not fail as often because it will get enough pixels. The change also includes a reduction in temporal filter permutation, generic denoiser handling arbitrary scales (not just hard coded halfres), proper history parameter, edge artifacts on final frame for ssgi, and avoiding ghosting when the resolution misses.
<img width="775" alt="image" src="https://media.github.cds.internal.unity3d.com/user/3327/files/0cde6d82-2176-412c-97ca-49279787f5d7">
@UnityAljosha UnityAljosha requested a review from a team as a code owner March 11, 2024 15:10
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).
See the PR template for more information.
Thank you!

@UnityAljosha UnityAljosha merged commit 6552c80 into 2022.3/staging Mar 13, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants