From 9880f1a9d2379f349156fdb93014ec6d0082d74c Mon Sep 17 00:00:00 2001 From: FrancescoC-Unity Date: Tue, 12 May 2020 17:24:42 +0200 Subject: [PATCH 1/2] Fix issue with alpha being over 1 hence, leading to the final color being boosted due to alpha handling --- .../Runtime/PostProcessing/Shaders/UberPost.compute | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/com.unity.render-pipelines.high-definition/Runtime/PostProcessing/Shaders/UberPost.compute b/com.unity.render-pipelines.high-definition/Runtime/PostProcessing/Shaders/UberPost.compute index 223aeabc3fc..8a0b252c5a1 100644 --- a/com.unity.render-pipelines.high-definition/Runtime/PostProcessing/Shaders/UberPost.compute +++ b/com.unity.render-pipelines.high-definition/Runtime/PostProcessing/Shaders/UberPost.compute @@ -218,7 +218,8 @@ void Uber(uint3 dispatchThreadId : SV_DispatchThreadID) // Alpha mask #ifdef ENABLE_ALPHA // Post processing is not applied on pixels with zero alpha - color.xyz = lerp(inputColor.xyz, color.xyz, inputColor.a); + // Saturate is necessary to avoid issues when additive blending pushes the alpha over 1. + color.xyz = lerp(inputColor.xyz, color.xyz, saturate(inputColor.a)); #endif // Done From 04f445e0e7c871c284edb76355cfbc7f333b2803 Mon Sep 17 00:00:00 2001 From: FrancescoC-Unity Date: Tue, 12 May 2020 17:24:54 +0200 Subject: [PATCH 2/2] Changelog --- com.unity.render-pipelines.high-definition/CHANGELOG.md | 1 + 1 file changed, 1 insertion(+) diff --git a/com.unity.render-pipelines.high-definition/CHANGELOG.md b/com.unity.render-pipelines.high-definition/CHANGELOG.md index e6d59ed0021..8d9a4af3076 100644 --- a/com.unity.render-pipelines.high-definition/CHANGELOG.md +++ b/com.unity.render-pipelines.high-definition/CHANGELOG.md @@ -581,6 +581,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0. - Fixed flickering of the game/scene view when lookdev is running. - Fixed issue with reflection probes in realtime time mode with OnEnable baking having wrong lighting with sky set to dynamic (case 1238047). - Fixed transparent motion vectors not working when in MSAA. +- Fixed issue with post process when running in RGBA16 and an object with additive blending is in the scene. ### Changed - Improve MIP selection for decals on Transparents