From 595535326d2bf99d2408237b441ea8b74cbc1abb Mon Sep 17 00:00:00 2001
From: Max Eskin <max@voxel8.com>
Date: Sat, 5 Dec 2020 19:05:41 -0500
Subject: [PATCH 1/2] Filter processor should use scaled vectors.

---
 .../Processing/Processors/Filters/FilterProcessor{TPixel}.cs  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/ImageSharp/Processing/Processors/Filters/FilterProcessor{TPixel}.cs b/src/ImageSharp/Processing/Processors/Filters/FilterProcessor{TPixel}.cs
index 4dc9e41960..d0c8ff40d7 100644
--- a/src/ImageSharp/Processing/Processors/Filters/FilterProcessor{TPixel}.cs
+++ b/src/ImageSharp/Processing/Processors/Filters/FilterProcessor{TPixel}.cs
@@ -72,11 +72,11 @@ public RowOperation(
             public void Invoke(int y, Span<Vector4> span)
             {
                 Span<TPixel> rowSpan = this.source.GetPixelRowSpan(y).Slice(this.startX, span.Length);
-                PixelOperations<TPixel>.Instance.ToVector4(this.configuration, rowSpan, span);
+                PixelOperations<TPixel>.Instance.ToVector4(this.configuration, rowSpan, span, PixelConversionModifiers.Scale);
 
                 ColorNumerics.Transform(span, ref Unsafe.AsRef(this.matrix));
 
-                PixelOperations<TPixel>.Instance.FromVector4Destructive(this.configuration, span, rowSpan);
+                PixelOperations<TPixel>.Instance.FromVector4Destructive(this.configuration, span, rowSpan, PixelConversionModifiers.Scale);
             }
         }
     }

From a311a164522b4f38cc02364572f68fba79f1a115 Mon Sep 17 00:00:00 2001
From: Max Eskin <max@voxel8.com>
Date: Mon, 7 Dec 2020 12:00:15 -0500
Subject: [PATCH 2/2] Add test.

---
 .../Processing/Filters/BrightnessTest.cs      | 33 +++++++++++++++++--
 1 file changed, 31 insertions(+), 2 deletions(-)

diff --git a/tests/ImageSharp.Tests/Processing/Filters/BrightnessTest.cs b/tests/ImageSharp.Tests/Processing/Filters/BrightnessTest.cs
index 75a9072c59..680a6afdce 100644
--- a/tests/ImageSharp.Tests/Processing/Filters/BrightnessTest.cs
+++ b/tests/ImageSharp.Tests/Processing/Filters/BrightnessTest.cs
@@ -1,6 +1,7 @@
-// Copyright (c) Six Labors.
+// Copyright (c) Six Labors.
 // Licensed under the Apache License, Version 2.0.
 
+using SixLabors.ImageSharp.PixelFormats;
 using SixLabors.ImageSharp.Processing;
 using SixLabors.ImageSharp.Processing.Processors.Filters;
 using Xunit;
@@ -26,5 +27,33 @@ public void Brightness_amount_rect_BrightnessProcessorDefaultsSet()
 
             Assert.Equal(1.5F, processor.Amount);
         }
+
+        [Fact]
+        public void Brightness_scaled_vector()
+        {
+            var rgbImage = new Image<Rgb24>(Configuration.Default, 100, 100, new Rgb24(0, 0, 0));
+
+            rgbImage.Mutate(x => x.ApplyProcessor(new BrightnessProcessor(2)));
+
+            Assert.Equal(new Rgb24(0, 0, 0), rgbImage[0, 0]);
+
+            rgbImage = new Image<Rgb24>(Configuration.Default, 100, 100, new Rgb24(10, 10, 10));
+
+            rgbImage.Mutate(x => x.ApplyProcessor(new BrightnessProcessor(2)));
+
+            Assert.Equal(new Rgb24(20, 20, 20), rgbImage[0, 0]);
+
+            var halfSingleImage = new Image<HalfSingle>(Configuration.Default, 100, 100, new HalfSingle(-1));
+
+            halfSingleImage.Mutate(x => x.ApplyProcessor(new BrightnessProcessor(2)));
+
+            Assert.Equal(new HalfSingle(-1), halfSingleImage[0, 0]);
+
+            halfSingleImage = new Image<HalfSingle>(Configuration.Default, 100, 100, new HalfSingle(-0.5f));
+
+            halfSingleImage.Mutate(x => x.ApplyProcessor(new BrightnessProcessor(2)));
+
+            Assert.Equal(new HalfSingle(0), halfSingleImage[0, 0]);
+        }
     }
-}
\ No newline at end of file
+}