Browse Source

Add Missing Transpose for Bradford matrix

pull/181/head
Tornhoof 9 years ago
parent
commit
b10287193c
  1. 2
      src/ImageSharp/Colors/Spaces/Conversion/Implementation/Lms/CieXyzAndLmsConverter.cs
  2. 2
      tests/ImageSharp.Tests/Colors/Colorspaces/RgbAndCieXyzConversionTest.cs

2
src/ImageSharp/Colors/Spaces/Conversion/Implementation/Lms/CieXyzAndLmsConverter.cs

@ -40,7 +40,7 @@ namespace ImageSharp.Colors.Spaces.Conversion.Implementation.Lms
/// </param>
public CieXyzAndLmsConverter(Matrix4x4 transformationMatrix)
{
this.TransformationMatrix = transformationMatrix;
this.TransformationMatrix = Matrix4x4.Transpose(transformationMatrix);
}
/// <summary>

2
tests/ImageSharp.Tests/Colors/Colorspaces/RgbAndCieXyzConversionTest.cs

@ -39,7 +39,7 @@
Rgb output = converter.ToRgb(input);
// Assert
Assert.Equal(output.WorkingSpace, Rgb.DefaultWorkingSpace);
Assert.Equal(output.WorkingSpace, Rgb.DefaultWorkingSpace); // TODO: Change Assert.Equal to the correct order, first the expected, then the current value
Assert.Equal(output.R, r, FloatComparerPrecision);
Assert.Equal(output.G, g, FloatComparerPrecision);
Assert.Equal(output.B, b, FloatComparerPrecision);

Loading…
Cancel
Save