Clamp is now set in gamma-space like threshold

Author: Chman

PostProcessing/Runtime/Effects/Bloom.cs

@@ -20,7 +20,7 @@ public sealed class Bloom : PostProcessEffectSettings
         [Range(0f, 1f), Tooltip("Makes transition between under/over-threshold gradual (0 = hard threshold, 1 = soft threshold).")]
         public FloatParameter softKnee = new FloatParameter { value = 0.5f };
 
-        [Tooltip("Clamps pixels to control the bloom amount.")]
+        [Tooltip("Clamps pixels to control the bloom amount. Value is in gamma-space.")]
         public FloatParameter clamp = new FloatParameter { value = 65472f };
 
         [Range(1f, 10f), Tooltip("Changes the extent of veiling effects. For maximum quality stick to integer values. Because this value changes the internal iteration count, animating it isn't recommended as it may introduce small hiccups in the perceived radius.")]
@@ -125,7 +125,8 @@ public override void Render(PostProcessRenderContext context)
             float knee = lthresh * settings.softKnee.value + 1e-5f;
             var threshold = new Vector4(lthresh, lthresh - knee, knee * 2f, 0.25f / knee);
             sheet.properties.SetVector(ShaderIDs.Threshold, threshold);
-            sheet.properties.SetVector(ShaderIDs.Params, new Vector4(settings.clamp, 0f, 0f, 0f));
+            float lclamp = Mathf.GammaToLinearSpace(settings.clamp.value);
+            sheet.properties.SetVector(ShaderIDs.Params, new Vector4(lclamp, 0f, 0f, 0f));
 
             int qualityOffset = settings.fastMode ? 1 : 0;
 

PostProcessing/Shaders/Builtins/Bloom.shader

@@ -23,8 +23,8 @@ Shader "Hidden/PostProcessing/Bloom"
         {
             half autoExposure = SAMPLE_TEXTURE2D(_AutoExposureTex, sampler_AutoExposureTex, uv).r;
             color *= autoExposure;
-            color = QuadraticThreshold(color, _Threshold.x, _Threshold.yzw);
             color = min(_Params.x, color); // clamp to max
+            color = QuadraticThreshold(color, _Threshold.x, _Threshold.yzw);
             return color;
         }