Skip to content

Remove function SimdHogExtractFeatures - #953

Merged
ermig1979 merged 1 commit into
devfrom
cursor/remove-hog-extract-features-e176
Sep 10, 2026
Merged

Remove function SimdHogExtractFeatures#953
ermig1979 merged 1 commit into
devfrom
cursor/remove-hog-extract-features-e176

Conversation

@ermig1979

@ermig1979 ermig1979 commented Sep 10, 2026

Copy link
Copy Markdown
Owner

Remove deprecated SimdHogExtractFeatures and its C++ wrapper Simd::HogExtractFeatures.

Changes

  • Remove the C API from SimdLib.h / SimdLib.cpp.
  • Remove the C++ wrapper from SimdLib.hpp.
  • Remove Base, SSE4.1, AVX2, AVX-512BW, NEON, and SVE2 implementations, including class HogFeatureExtractor.
  • Remove AVX-512BW CovertDifference and SVE2 HogTailMask / HogDifferenceLo / HogDifferenceHi, which were only used by HogExtractFeatures.
  • Keep other Hog APIs (HogDeinterleave, HogFilterSeparable) and their *Hog.cpp translation units.
  • Remove HogExtractFeatures auto-tests from TestHog.cpp / Test.cpp.
  • Update release notes for 7.2.166 in docs/2026.html.

Python wrapper / VS2022

  • The Python wrapper does not expose SimdHogExtractFeatures, so py/SimdPy is unchanged.
  • prj/vs2022 project files are unchanged because Hog source files still contain HogDeinterleave and HogFilterSeparable.

Testing

Release CMake build with g++ (-DSIMD_SHARED=ON):

  • nm -D libSimd.so exports SimdHogDeinterleave and SimdHogFilterSeparable only (SimdHogExtractFeatures is gone).
  • ./Test "-r=.." -fi=Hog -tt=1 -ts=1 passed (HogDeinterleave, HogFilterSeparable).
  • ./Test "-r=.." -fi=HogExtractFeatures reports no matching tests.
  • ./Test "-r=.." -fi=Sobel -tt=1 -ts=1 passed.
  • python3 ./Test.py -r=.. -i PrintInfo and -i ImageFill completed successfully.
  • GitHub CI: build_and_test_gcc_new and build_and_test_arm64 succeeded.
Open in Web Open in Cursor 

Drop the C API, C++ wrapper, Base/SSE4.1/AVX2/AVX-512BW/NEON/SVE2
implementations, HogFeatureExtractor, unused Hog helpers, and related tests.

Co-authored-by: igor.ermolaev <igor.ermolaev@altezza.org>
@ermig1979
ermig1979 marked this pull request as ready for review September 10, 2026 06:02
@ermig1979
ermig1979 merged commit 8ec85f9 into dev Sep 10, 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