Skip to content

fix: ignore* option parity with resemble (pixelmatch)#1181

Open
wswebcreation wants to merge 4 commits into
mainfrom
ws-ignore-feature-parity
Open

fix: ignore* option parity with resemble (pixelmatch)#1181
wswebcreation wants to merge 4 commits into
mainfrom
ws-ignore-feature-parity

Conversation

@wswebcreation

Copy link
Copy Markdown
Member

Summary

Integration PR for v10 pixelmatch ignore* parity with resemble.js. Merges four feature branches from ws-ignore-feature-parity:

Builds on #1175 (ignoreAntialiasing toggle, default true).

Behaviour highlights

Scenario Result
No explicit ignore* config Forgiving default unchanged (ignoreAntialiasing: true)
ignoreLess + default antialiasing Last-wins → strict AA (bug fix)
ignoreColors hue-only diff Pass (resemble brightness parity)
Multi-flag combos Last-wins order: alphaantialiasingcolorslessnothing

- apply last-wins threshold and AA rules for ignoreLess, ignoreAlpha, ignoreColors, and ignoreNothing
- keep forgiving default when only ignoreAntialiasing applies
- add unit and executeImageCompare integration tests
- add patch changeset for image-comparison-core and visual-service
- replace BT.601 grayscale preprocessing in compareImages
- add unit and integration tests for ignoreColors parity
- add patch changeset for image-comparison-core and visual-service
- add PNG fixture pairs for each ignore* scenario from the parity plan
- add ignore-options.parity.test.ts with expected pass/fail matrix
- document fixtures and regeneration steps in README
- add JSDoc preset mapping on each ignore* option
- add README section with comparison table and last-wins semantics
- remove redundant ignoreAntialiasing from Sauce/Lambda configs
- add patch changeset
@changeset-bot

changeset-bot Bot commented Jul 5, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: f7ff230

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@wdio/image-comparison-core Patch
@wdio/visual-service Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant