Normalize image color spaces before comparison#665
Conversation
Use the sRGB converted snapshot when doing the perceptual comparison This reduces the chances of failures when comparing snapshots in different color spaces
|
Hey. Should we merge this PR after the rebase? Looks like it was approved, but never merged |
|
@stephencelis Please, is there a plan to merge this? |
# Conflicts: # Sources/SnapshotTesting/Snapshotting/NSImage.swift # Sources/SnapshotTesting/Snapshotting/UIImage.swift # Tests/SnapshotTestingTests/SnapshotTestingTests.swift
|
@stephencelis I will bump this topic cause there was no answer from few months Could you provide information what is blocking you from merging it into the main branch? It will be nice feature/fix that could reduce lots of issues. Thanks! |
|
+1 please merge :) |
|
After a while I figured out the difference between my CI machine and the dev machine was the color space, running on macOS. |
# Conflicts: # Tests/SnapshotTestingTests/SnapshotTestingTests.swift
|
Hi @mbrandonw or @stephencelis, would one of you be able to look at this PR? It has been waiting for a while and addresses issues when collaborating with people using various hardware to run snapshot tests. At least six other people (unaffiliated with me) stated they are experiencing these issues. |
|
I'm also experiencing this issue, would love to see this merged @stephencelis @mbrandonw |
|
I am experiencing these issues as well, hoping to see this merged soon. @mbrandonw & @stephencelis any updates? |
|
We're experiencing the same issue. Hoping this will be resolved soon by normalizing the snapshot colors—this change is definitely needed. Thanks! |
|
Any chance to check on conflicts and merge these? I am experiencing these issues too. |
Overview
Use the sRGB converted snapshot when doing the perceptual comparison. This reduces the chances of failures when comparing snapshots using different color spaces.
This colorspace normalization technique was originally introduced in #446 and this PR extends it to both the reference and new images when performing perceptual image comparison.
Unit tests were added to verify that images in the P3 and sRGB colors spaces match after colorspace normalization.
Related Issues