Encoding Results in Washed-Out Colors

[lucasfischer]

macOS: 14.1
Xcode 15.0
SDWebImage 5.18.4
SDWebImageWebPCoder 0.14.1

I tried to implement this package into an existing macOS app, when I noticed that the conversion results in washed-out colors. I tried this with converting PNGs and JPGs to WebP so far. Are there any options to preserve the original image colors?

Tried many different option combinations, but the images in the ZIP were created with .encodeCompressionQuality = 0.8 and an NSImage.

Thank you for working on this project.
I hope you are having a nice day.

examples.zip

[dreampiggy]

conversion results in washed-out colors

Seems Bug of color space decoding/encoding. Any demo ?

And I remember, the encode of WebP does not support some color format. In history we use the vImage (vImageConvert_AnyToAny) to convert like ARGB8 -> RGB8 (BGRA8 -> RGB8) to use libwebp. But after that we revert back to use libwebp's own processing. Seems bug maybe related to this.

[dreampiggy]

Seems not bug of color format (which means, RGBA or ARGB or BGRA, the order)

Seems bug of color space, like sRGB, P3, PQ

[dreampiggy]

Where's your source PNG image ? (You means, you convert PNG to JPEG and WebP. The converted JPEG and WebP use different color space ?)

[dreampiggy]

I guess this cause issue. The color space should not use device RGB

Should use CGImage.colorspace. Which means, input image (from PNG) may contains color space.

[dreampiggy]

I'm sure by changing the color space to use CGImage's color space, the issue been fixed.

pexels-egor-kamelev-920163.zip

See:

[lucasfischer]

Thank you for this quick reply!

Seems like your instinct with it being an issue with the color space has been spot on.
I downloaded your ZIP file and compared the files in them. It looks like it's fixed!

You are so fast. Thank you so much for looking into this. ❤️

[dreampiggy]

Just use that PR's commits and test by yourself.

Then I'll merge in and release a patch version

[lucasfischer]

I just tried the PR and for most of my tests it spits out perfect results. 🤩

Although, the problem still persists for all HEIC files from my iPhone and for one of the PNGs I tested.

Do you have another quick idea?

examples.zip

[dreampiggy]

For you case, actually we should not only ensure the WebP encoding process, also need to ensure The decoded image, like HEIF, use the correct color space.

Because it's "Transcoding", right ?

Actually, it do 3 steps:

1. Decode original data into image buffer (will this contains bug? HEIF Decoder ? PNG Decoder)
2. Encode the image buffer into new data (will this contains bug? WebP Encoder ?) // -> I fixed this
3. Decode the new data into new image buffer and render (will this contains bug? WebP Decoder ?)

You want the image buffer in 1. match the new image buffer in 3., so all of thesse steps need to check again

[dreampiggy]

How do you get the original input image, using the SDWebImage's API (NSImage.sd_imageWithData) or AppKit API (like NSImage(data:)` ?

Which means, which decoder of input image you used.

[dreampiggy]

png.zip

- (void)viewDidLoad {
    [[SDImageCodersManager sharedManager] addCoder:[SDImageWebPCoder sharedCoder]];
    
    NSString *inputName = @"png/before.png";
    NSString *inputPath = [NSBundle.mainBundle pathForResource:inputName ofType:nil inDirectory:@"examples"];
    NSData *inputData = [NSData dataWithContentsOfFile:inputPath];
    UIImage *inputImage = [[UIImage alloc] initWithData:inputData];
    
    NSData *outputData = [[SDImageWebPCoder sharedCoder] encodedDataWithImage:inputImage format:SDImageFormatWebP options:nil];
    [outputData writeToFile:@"/tmp/output.webp" atomically:YES];
}

Using AppKit's PNG decoder seems work well. Is this SDWebImage's PNG decoder's bug ?

[dreampiggy]

output-PNG-decoder.zip

Even use the UIImage sd_imageWithData: to decode the PNG from your input, the result is the same

I guess is your App's business code, which does not give the correct input (before feeding into SDImageWebPCoder), check your logic 's bug?

You can even use LLDB debugger, to visualize the image object (NSImage) in memory and preview.app, is that match what you see in Mac's Finder ?