Skip to content

diffimg: binary tool for comparing images#9668

Merged
poweifeng merged 3 commits intomainfrom
pf/imgdiff-tool
Feb 10, 2026
Merged

diffimg: binary tool for comparing images#9668
poweifeng merged 3 commits intomainfrom
pf/imgdiff-tool

Conversation

@poweifeng
Copy link
Contributor

This tool uses existing libraries: image, imageio, imageio-lite, imagediff to perform difference comparison for on-disk images.

We refactor renderdiff to use this tool instead of using python dependencies.

@poweifeng poweifeng added the internal Issue/PR does not affect clients label Jan 30, 2026
@poweifeng poweifeng force-pushed the pf/imgdiff-tool branch 2 times, most recently from a2a72e8 to eb74b90 Compare January 31, 2026 00:17
This tool uses existing libraries: image, imageio, imageio-lite,
imagediff to perform difference comparison for on-disk images.

We refactor renderdiff to use this tool instead of using
python dependencies.
std::string ext = path.getExtension();
std::transform(ext.begin(), ext.end(), ext.begin(), ::tolower);

bool preferLite = (ext == "tif" || ext == "tiff");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's the benefit of using imageio lite?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, this can read a bit strange. So I tried to list reasons in the initial commit:
b40530a

poweifeng and others added 2 commits February 10, 2026 14:22
Co-authored-by: Ben Doherty <bendoherty@google.com>
@poweifeng poweifeng enabled auto-merge (squash) February 10, 2026 22:25
@poweifeng poweifeng merged commit d6caa9d into main Feb 10, 2026
17 checks passed
@poweifeng poweifeng deleted the pf/imgdiff-tool branch February 10, 2026 22:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Issue/PR does not affect clients

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants