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 #7978

Merged
merged 8 commits into from
Oct 16, 2023
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.

pjbunity and others added 8 commits October 10, 2023 12:51
…acing light list

This is a backport based on https://github.cds.internal.unity3d.com/unity/unity/pull/35672 for branch 2022.3/staging.

This PR is in response to this bug-report: https://jira.unity3d.com/browse/UUM-47230

The artefacts reported are caused by a limit on the maximum number of lights in local neighborhoods in the path tracer. This PR allows users to change this limit through [the shader config mechanism](https://docs.unity3d.com/Packages/[email protected]/manual/HDRP-Config-Package.html).
Fix unexpected missing refraction property in some case of VFX.

Issue introduced at https://github.cds.internal.unity3d.com/unity/unity/commit/7986ee8ea4efd9647658ebfd201837e76f6716ef
As commented in code, this special additional check is only needed because VFX is taking a shortcut.
gathering of PRs

Improve computation of hashCode for animationCurves (#35356)
Add warning message in light cluster override UI if raytraycing is off (#35343) 
Fixed an issue where non directional light could react to "interact with sky" flag. (#35349) 
[HDRP] HDRP Wizard Warning Non-Built-In Materials (#35263) 
ensure camera are valid before destroying (#35491)
Fix the SetData error by adding a max light count + by fixing the Assert (wasn't working) (#35305)
fix refcounting and cleanup of skyContext when renderer changes (#35437)
[HDRP] Fixed cascaded shadowmaps cascade borders and blending issue with shadow mask (#35432)
Fixed Layered Lit Displacement
Fix for this issue: https://jira.unity3d.com/browse/UUM-52909.

Due to some perf improvements and caching, a previous comparison by reference was no longer valid in undo/redo (which undergoes serialization). This quick fix compares properties by ID instead of reference.
Backport https://jira.unity3d.com/browse/UUM-38845

The original artefact that caused the test to be disabled on PS5 did not reproduce anymore. Instead a new bug appeared which is related to the computation of noise values on PS5. The PSSL shader compiler was unhappy to divide by float(0xFFFFFFFF), and failed to generate any noise pattern. The workaround is to replace float(0xFFFFFFFF) by float(0x00FFFFFF), which is the maximal continuous integer value that can be represented as a float (mantissa is 24 bits).

Contrary to other backports for this bug, 010-BRG-Simple does not exist on this branch and is thus not an instability.

Reference image for PS5 HDRP Runtime test 009-SG-FullScreenTarget was incorrect and did not have procedural noise working: new reference image fixes this.
@UnityAljosha UnityAljosha requested a review from a team as a code owner October 16, 2023 09:41
@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).
See the PR template for more information.
Thank you!

@UnityAljosha UnityAljosha merged commit 9a3412a into 2022.3/staging Oct 16, 2023
3 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