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

Universal Staging Branch #6266

Merged
merged 11 commits into from
Nov 16, 2021
Merged

Universal Staging Branch #6266

merged 11 commits into from
Nov 16, 2021

Conversation

phi-lira
Copy link
Contributor

@phi-lira phi-lira commented Nov 10, 2021

Purpose of this PR

skerys and others added 11 commits November 10, 2021 09:24
#5509)

* Moving WS view direction from vertex to fragment to lower varying size

* Removed new viewDirectionWS varying references
* switching _CameraDepthTexture to R32Float and accommodating some stuff for that

* some changes to the Foundation Project for Repro project (to be reverted later on)

* Revert "some changes to the Foundation Project for Repro project (to be reverted later on)"

This reverts commit 80af9be.

* disable fbfetch on DbufferDeferred decals

* adding IsTransientAttachment to ScriptableRenderPass/NativeRenderPass code

* Fixed decals to work with native render pass [case 1353141]

* Adding changelog entry

* Leftover

* making depthOnly pass compatible to render color texture and changing shaders to accommodate that

* Fixing decal+native renderpass+decal+deferred scenario

* fix after updating trunk

* merge fix - change descriptor for DepthOnly configureTarget to the correct one

* enabling depthbuffer for depth copy and adding additional override for ConfigureTarget

* formatting fixes

* terrain shader passes depthonly pass fix to output clippos

* adding depth copy for renderpass as well and changing how fake depth is handled

* some cleanup and refactoring and enabling Renderpass for SS forward decals

* Fixing decals rendering to be correct with render scale. [1353885]

* Cleanup

* Cleanup

* Cleanup

* adding _OUTPUT_DEPTH keyword to CopyDepthPass to select correct output buffer in shader

* adding depth output for VFXGraph depth only passes

* changing the semantic for outDepthColor in VFXPasses.template

* Changing to correct normal sampling for decal shadergraph

* Changing image to correct texture type

* fixing DBuffer Decals CopyDepth pass not copying the depth and failing rendering

* adding fb fetch to Gbuffer decals and some other depth related fixes for decals

* fixing shader compilation error and renaming define for fb fetch idx

* Adding normal alpha multiplication with albedo alpha multiplication

* fix for RTIs non render pass platforms

* another DeferredLights null check

* null exception fix

* adding some editor-specific code to avoid a bug with editor setting incorrect depth buffer while on game view

* Updating terrain 230 test templates from extra

* Updating test 230 images

* Updating 230 vulkan images

Co-authored-by: Jonas Sideravicius <jonass@unity3d.com>
Co-authored-by: Felipe Lira <felipedrl@gmail.com>
* Fixed issue where global lights with shadows enabled would break batching

* Added layer batching test

* Updated test camera (to remove transparency) and added test images

* Updated changelog
… same way as the swapchain backend implementation (#6180)

* made the Vulkan MSAA support fallback behaviour behave in the same way as the swapchain's backend code

* added comments to the Vulkan MSAA fallback code

* Added changelog entry

Co-authored-by: Felipe Lira <felipedrl@gmail.com>
#6219)

* Fix regression where discard wasn't getting properly called in a 2D shader.

- Calling AlphaDiscard here is wrong because it requires _ALPHATEST_ON keyword to be enabled to work, and this keyword isn't used in 2D shaders.

* Changelog

Co-authored-by: Felipe Lira <felipedrl@gmail.com>
This is necasary for custom functions and some other nodes
Co-authored-by: Felipe Lira <felipedrl@gmail.com>
…etting (#6192)

* Make LensFlare dimensions respect RenderScale setting (also passing CameraData instead of Camera)

* using descriptor for lensflare descriptions instead of multiplying by renderscale

* Update CHANGELOG.md

Co-authored-by: Felipe Lira <felipedrl@gmail.com>
* Remove conditional on shadow offset constants.

* Add change log.

Co-authored-by: Felipe Lira <felipedrl@gmail.com>
@phi-lira phi-lira requested review from a team as code owners November 10, 2021 10:04
@github-actions
Copy link

Hi! This comment will help you figure out which jobs to run before merging your PR. The suggestions are dynamic based on what files you have changed.
Link to Yamato: https://unity-ci.cds.internal.unity3d.com/project/902/
Search for your PR branch using the search bar at the top, then add the following segment(s) to the end of the URL (you may need multiple tabs depending on how many packages you change)

URP
/jobDefinition/.yamato%252Fall-urp.yml%2523PR_URP_trunk
With changes to URP packages, you should also run
/jobDefinition/.yamato%2Fall-lightmapping.yml%23PR_Lightmapping_trunk

Shader Graph
/jobDefinition/.yamato%252Fall-shadergraph.yml%2523PR_ShaderGraph_trunk
Depending on your PR, you may also want
/jobDefinition/.yamato%252Fall-shadergraph_builtin_foundation.yml%2523PR_ShaderGraph_BuiltIn_Foundation_trunk
/jobDefinition/.yamato%252Fall-shadergraph_builtin_lighting.yml%2523PR_ShaderGraph_BuiltIn_Lighting_trunk

Depending on the scope of your PR, you may need to run more jobs than what has been suggested. Please speak to your lead or a Graphics SDET (#devs-graphics-automation) if you are unsure.

@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!

Copy link

@ernestasKupciunas ernestasKupciunas left a comment

Choose a reason for hiding this comment

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

No issues were found regards the staging branch itself. Tested locally using 2 projects: URP template scene and Winter project.

PRs that needs some improvements:
#6167 << no description at all
#6223 << needs tiny improvements: no FB case linked in the description, “Ran tests locally.” - needs clarification.

Other than that, it's a green light

@phi-lira
Copy link
Contributor Author

phi-lira commented Nov 16, 2021

Universal_Stereo on Win__Standalone_mono_Linear on version trunk failing on master 🔴

URP_PostPro on Android_Vulkan_Standalone_il2cpp_Linear on version trunk failing on master 🔴

URP_PostPro on Android_OpenGLES3_Standalone_il2cpp_Linear on version trunk - 🟡 appears to be introduced by this PR. Bisecting

URP_Terrain on OSX_Metal_playmode_mono_Linear on version trunk - timeout issue that fails to output UnityLog. Cannot check if there's an issue introduced in this PR 🟡

URP_Performance_LostCrypt on OSX_Metal_performance_build_editmode_mono - crash, also seen in master, checking for instability.

URP_Performance_LostCrypt on OSX_Metal_performance_build_editmode_mono - 🔴 also seen in master

URP_Lighting on OSX_Metal_playmode_mono_Linear on version trunk - timeout issue that fails to output UnityLog. Cannot check if there's an issue introduced in this PR 🟡

Split Projects are all instability issues...

@phi-lira phi-lira merged commit e9cebe9 into master Nov 16, 2021
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.