Skip to content

Conversation

AndrewSaraevUnity
Copy link
Contributor

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

This adds DOTS_INSTANCING_ON variants to the "HDRP/Decal" shader passes used to render mesh decals, which allows them to be rendered with Hybrid Renderer V2.


Testing status

Tested manually with a SubScene in the editor 2021.2.0a14.1979 with Hybrid Renderer 0.11.0-preview.42.


Comments to reviewers

This is mainly required in 2020.2/partner/staging branch but since this change doesn't depend on the partner editor it might be better to backport it instead of forward-porting it later.

…rt mesh decals using it in the Hybrid Renderer.
@sebastienlagarde sebastienlagarde marked this pull request as ready for review April 22, 2021 21:24
@sebastienlagarde sebastienlagarde merged commit fa2ff6e into master Apr 22, 2021
@sebastienlagarde sebastienlagarde deleted the hybrid/fix-mesh-decal-shader branch April 22, 2021 21:25
sebastienlagarde added a commit that referenced this pull request Apr 22, 2021
…mesh decals using it in the Hybrid Renderer. (#4284)pp
sebastienlagarde added a commit that referenced this pull request Apr 22, 2021
* Merge Hd/bugfix #4265

* fix compil issue

* fix compil issue

* Fix shader warning when using Lanczos upsampling (#4280)

* Updated ray tracing settings and added it to the table of contents (#4279)

* Added DOTS_INSTANCING_ON variants to the "HDRP/Decal" shader to surt mesh decals using it in the Hybrid Renderer. (#4284)pp

* Update decal stress test reference screenshots for vulkan

Co-authored-by: Pavlos Mavridis <pavlos.mavridis@unity3d.com>
Co-authored-by: Lewis Jordan <lewis.jordan@hotmail.co.uk>
AndrewSaraevUnity added a commit that referenced this pull request Apr 29, 2021
…rt mesh decals using it in the Hybrid Renderer. (#4284)

(cherry picked from commit fa2ff6e)
sebastienlagarde added a commit that referenced this pull request May 1, 2021
* [10.x.x HDRP] Fix issue with raytracing resources not properly initialize (#4268)

* draft

* Fix build DXR

* Match behavior with the one in master

* Updated ray tracing settings and added it to the table of contents  (#4279)

* Updated ray tracing settings and added it to the toc

* Removed hyphens from rtss toc entry

* Fix shader warning when using Lanczos upsampling (#4280)

* Added DOTS_INSTANCING_ON variants to the "HDRP/Decal" shader #4284

* [HDRP] Fp16_PPAlpha Panini projection, Motion Blur test and moved other Fp16Alpha scenes to one #4290

* [HDRP] Fix undo of some properties on light editor #4293

* Opt-out of builtin auto baking of ambient/reflection probe. #4324

* Revert "Opt-out of builtin auto baking of ambient/reflection probe. #4324"

This reverts commit 719e835.

* Hd/redo fix inputregistering domainreload (#4358)

* Update InputRegistering.cs

* Remove asset modification if InputSystem package is in use

* Prevent missuse of InputRegistering while InputSystem package is in use

* cleaning

* more cleaning

* Update code to use Next instead of GetArrayElementAtIndex

* Fix strange out of range error

Co-authored-by: Lewis Jordan <lewis.jordan@hotmail.co.uk>
Co-authored-by: Pavlos Mavridis <pavlos.mavridis@unity3d.com>
Co-authored-by: Andrew Saraev <67020478+AndrewSaraevUnity@users.noreply.github.com>
Co-authored-by: TomasKiniulis <50582134+TomasKiniulis@users.noreply.github.com>
Co-authored-by: Adrien de Tocqueville <adrien.tocqueville@unity3d.com>
Co-authored-by: JulienIgnace-Unity <julien@unity3d.com>
Co-authored-by: Remi Slysz <40034005+RSlysz@users.noreply.github.com>
AndrewSaraevUnity added a commit that referenced this pull request May 4, 2021
* [HDRP] Make Decal shader compatible with SRP batcher (#3806)

* Fix SRP batcher with decal

* Update CHANGELOG.md

* Update CHANGELOG.md

(cherry picked from commit 5a88bb8)

* Added DOTS_INSTANCING_ON variants to the "HDRP/Decal" shader to support mesh decals using it in the Hybrid Renderer. (#4284)

(cherry picked from commit fa2ff6e)

* [2020.2] [10.x.x] [Bugfix 1296776] Enforce SRP Batcher and Hybrid renderer compatibility by always using a consolidated property list (#3577)

* Make all passes in a subshader use the same HLSL proeprty declarations

* Adding changelog

* Better fix for Targets accessing the current blocks

(cherry picked from commit 26a1096)

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>
Co-authored-by: Chris Tchou <ctchou@unity3d.com>
pastasfuture pushed a commit that referenced this pull request Aug 20, 2021
* [HDRP] Make Decal shader compatible with SRP batcher (#3806)

* Fix SRP batcher with decal

* Update CHANGELOG.md

* Update CHANGELOG.md

(cherry picked from commit 5a88bb8)

* Added DOTS_INSTANCING_ON variants to the "HDRP/Decal" shader to support mesh decals using it in the Hybrid Renderer. (#4284)

(cherry picked from commit fa2ff6e)

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>
pastasfuture pushed a commit that referenced this pull request Sep 15, 2021
* [HDRP] Make Decal shader compatible with SRP batcher (#3806)

* Fix SRP batcher with decal

* Update CHANGELOG.md

* Update CHANGELOG.md

(cherry picked from commit 5a88bb8)

* Added DOTS_INSTANCING_ON variants to the "HDRP/Decal" shader to support mesh decals using it in the Hybrid Renderer. (#4284)

(cherry picked from commit fa2ff6e)

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>
mohammad22 pushed a commit that referenced this pull request Mar 2, 2022
* [HDRP] Make Decal shader compatible with SRP batcher (#3806)

* Fix SRP batcher with decal

* Update CHANGELOG.md

* Update CHANGELOG.md

(cherry picked from commit 5a88bb8)

* Added DOTS_INSTANCING_ON variants to the "HDRP/Decal" shader to support mesh decals using it in the Hybrid Renderer. (#4284)

(cherry picked from commit fa2ff6e)

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>
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.

2 participants