Skip to content

Fix RenderQueue generation for HDRP code #691

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Jun 3, 2020
Merged

Fix RenderQueue generation for HDRP code #691

merged 5 commits into from
Jun 3, 2020

Conversation

sebastienlagarde
Copy link
Contributor

Please read

PR workflow guidelines

  • SRP ABV will start automatically on Yamato when you open your PR
  • Changes to docs and md files will not trigger ABV jobs
  • Consider making use of draft PRs if you are not 100% sure that your PR is ready for review
  • ABV will restart if you add a new commit to a branch with an open PR (hence why you should consider using draft PRs)
  • Adding [skip ci] (case insensitive) to the title of PRs will stop any jobs being trigger automatically - you will need to open Yamato and find your branch to run ABV
  • You can also add [skip ci] to commit messages to prevent CI from running on that push
  • Add [cancel old ci] to your commit message if you've made changes you want to test and no longer need the previous jobs

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.
  • 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

Manual Tests: What did you do?

  • Opened test project + Run graphic tests locally
  • Built a player
  • Checked new UI names with UX convention
  • Tested UI multi-edition + Undo/Redo + Prefab overrides + Alignment in Preset
  • C# and shader warnings (supress shader cache to see them)
  • Checked new resources path for the reloader (in developer mode, you have a button at end of resources that check the paths)
  • Other:

Automated Tests: What did you setup? (Add a screenshot or the reference image of the test please)

Yamato: (Select your branch):
https://yamato.prd.cds.internal.unity3d.com/jobs/902-Graphics

Any test projects to go with this to help reviewers?

Comments to reviewers

Notes for the reviewers you have assigned.

@sebastienlagarde sebastienlagarde requested a review from alelievr June 1, 2020 22:13
@github-actions github-actions bot added the HDRP label Jun 1, 2020
@sebastienlagarde sebastienlagarde requested a review from a team as a code owner June 3, 2020 12:14
@sebastienlagarde sebastienlagarde requested review from martint-unity and removed request for a team and martint-unity June 3, 2020 12:14
@sebastienlagarde sebastienlagarde merged commit cbf8377 into sg/master-stack Jun 3, 2020
@sebastienlagarde sebastienlagarde deleted the HDRP/sg-master-stack-renderingpass branch June 3, 2020 13:02
sebastienlagarde added a commit that referenced this pull request Jun 3, 2020
@sebastienlagarde
Fix RenderQueue generation for HDRP code (#691)
fd7a1d0 
* Fix renderqueu generation

* fix renderingPass correctly

* clean m_AddPrecomputedVelocity usuage
sebastienlagarde added a commit that referenced this pull request Jun 10, 2020
@sebastienlagarde @alelievr
@alindmanUnity @elizabeth-legros @marctem
[10.x.x] SG Master stack + HDRP sub targets refactor [Skip CI] (#797)
a9230dd 
* Merging into master:   squash merge sg/master-stack into 9.x.x/release

Effectively taking the diff between
   sg/master-stack commit: b6b5b1f
   9.x.x/release commit:  808d18e
and applying it to master

# Conflicts:
#	com.unity.render-pipelines.universal/Editor/ShaderGraph/Includes/SpriteForwardPass.hlsl
#	com.unity.render-pipelines.universal/Editor/ShaderGraph/Includes/SpriteLitPass.hlsl
#	com.unity.render-pipelines.universal/Editor/ShaderGraph/Includes/SpriteUnlitPass.hlsl
#	com.unity.shadergraph/Editor/Drawing/Views/MaterialNodeView.cs
#	com.unity.shadergraph/Editor/Drawing/Views/PortInputView.cs

* Add  protection in migration when rendering pass was set to background (#703)

* Add  protection in migration when rendering pass was set to background

* Fix rendering pass migration for unlit

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>

* Fix RenderQueue generation for HDRP code (#691)

* Fix renderqueu generation

* fix renderingPass correctly

* clean m_AddPrecomputedVelocity usuage

* update edge type converstion test to stacks

* change context to take in a tuple of (blockdescriptor descriptor,bool isDefaultValue)

* Adding mesh deformation nodes to the BuiltIn node type list

* Added VT TextureStack.hlsl include to RT passes, and fixed a shader warning spam

* connected slots are not default value

* Fix for DOTS hybrid skinning in HDRP

* Add settings for Virtual Texture Node back to Graph Inspector, fix a layout issue.

* HDRP sub targets refactor (#773)

* Moved sub-target upgrade code to migration files and refactored the HD blocks

# Conflicts:
#	TestProjects/HDRP_Tests/Assets/GraphicTests/Scenes/9x_Other/9701_CustomPass_DrawRenderers/9701_UnlitRT.mat

* Made property row labels flexible

* Fixed indent level of propertyRow

* Added more control to indent level and clear foldout

* Refactored inspector GUI for HD lit and hair master node

* Fixed compilation

* Moved fabric subtarget GUI to new system

* Converted Eye and stacklit settings to new GUI system

* Finished decal and unlit inspector settings

* Add missing unlit distortion only + remove unlitPass template + fix duplicated alpha to mask option

* Testing new pass override system

* Revert assets

* Started to factorize passes for Unlit & fix distortion only in the inspector UI

* Shared all non DXR passes between shaders

* Fixed unlit depth prepass

* Fix lot of issues with shared render passes

* Moved SSS property to target specific and converted all ray tracing passes

* Fix missing normal space

* Shared more code between passes

* Added file dependency and remove unused template path

* Added target block context changes

* Removed all the commented code

* Fix anisotropy indent level

* Fixed yamato compilation issue

* Fixed stacklit and raytracing prepass shader issues

* Invert pass order

* Fixed shadow matte

* Enable virtual texturing in forward only

* Wrong emplacement for virtual texture feedback

* Fixed unwanted require field list modification

* Update 801_SubSurfaceScatteringDeferred.png

* Fix vulkan and metal

* Revert "Fix vulkan and metal"

This reverts commit 367ad17.

* Update HDShaderPasses.cs

* Update materials

* Update materials part 2 HDRP-test

* Update material for DXR projects

* update VFX project material

Co-authored-by: Chris Tchou <ctchou@unity3d.com>
Co-authored-by: Antoine Lelievre <antoinel@unity3d.com>
Co-authored-by: Alex Lindman <alexandral@unity3d.com>
Co-authored-by: Elizabeth LeGros <elizabeth.legros@unity3d.com>
Co-authored-by: Marc Templin <marcte@unity3d.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alelievr alelievr alelievr approved these changes

Assignees
No one assigned
Labels
HDRP
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sebastienlagarde @alelievr