Skip to content

Improve description of structure Simd::ImageMatcher - #947

Merged
ermig1979 merged 1 commit into
devfrom
cursor/improve-image-matcher-docs-9c50
Sep 4, 2026
Merged

Improve description of structure Simd::ImageMatcher#947
ermig1979 merged 1 commit into
devfrom
cursor/improve-image-matcher-docs-9c50

Conversation

@ermig1979

Copy link
Copy Markdown
Owner

Summary

Improved Doxygen descriptions of structure Simd::ImageMatcher in src/Simd/SimdImageMatcher.hpp from real usage in src/Use/UseImageMatcher.cpp (report all similar pairs) and src/Test/TestImageMatcher.cpp (filter unique images). Added the change to release 7.2.166 notes in docs/2026.html.

The comments now describe:

  • Reduced-image matching (16x16 / 32x32 / 64x64) with a 4x4 fast hash pre-filter and normalized RMS difference
  • Typical InitCreateFindAdd flow, with Tag as a source index (size_t)
  • Two usage patterns: filter unique images (Add only when Find is false) and report all similar pairs (always Add, read Result::hash->tag and difference * 100)
  • Init matcher selection from number, threshold and normalized
  • Create converting non-Gray8 to Gray8 and resizing with SimdResizeMethodArea
  • Correct Empty() meaning (true before Init) and the Hash64x64 size (was wrongly documented as 32x32)
  • Skip() excluding a stored hash from later Find without removing it

No executable code was changed. The test suite was not run, as requested.

Files

  • src/Simd/SimdImageMatcher.hpp
  • docs/2026.html (release 7.2.166)

Updated ImageMatcher comments
Release 7.2.166 notes

To show artifacts inline, enable in settings.

Open in Web Open in Cursor 

Document ImageMatcher from real usage: Init/Create/Find/Add flow,
filter-unique vs report-all-pairs patterns, Hash tag as source index,
and the Hash64x64 / Empty() comment fixes.

Co-authored-by: igor.ermolaev <igor.ermolaev@altezza.org>
@ermig1979
ermig1979 marked this pull request as ready for review September 4, 2026 15:04
@ermig1979
ermig1979 merged commit 9e89020 into dev Sep 4, 2026
2 checks passed
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.

2 participants