Patching test_imgwarp is removed, because Imgproc_RemapRelative test
is also removed from the list, since WARP_RELATIVE_MAP is
not implemented in KleidiCV.

Yufeng Shi authored Jan 29, 2025 and Denes Tarjan committed Feb 06, 2025

 1. Refactor RemapF32 and Warp_Perspective SVE2 Implementation
 2. Enable constant border for RemapF32 Neon Implementation
 3. Refactor test_remap.cpp and add tests for constant border

* Warn if features are only supported with patched OpenCV.
* Clarify what is meant by "in-place"
* In-place *is* supported for resize, transpose & Sobel because OpenCV
  will create a new image before calling the HAL.
* Note optimal sigma values for Gaussian blur.
* Note the need to specify accuracy hint for Gaussian blur.
* Add missing constraints on element size for transpose & rotate.
* Fix references to non-existent src & dst parameters.
* Use Mat's type instead of both depth and channels.
* Remove confusing explanation of remap's fixed point format and
  instead reference OpenCV's docs.
* sum is disabled by default so remove it from the docs.
* ^ means bitwise xor in C, so don't use it to mean power.

Instead set cv::GaussianBlur's hint parameter to approximate.
In addition, change build scripts to use the approximate hint by
default. This is necessary to ensure that OpenCV's tests and benchmarks
will use KleidiCV's Gaussian blur.

Array2D used to initialize the allocated memory to all zeros by
default, but that behavior was removed for performance reasons.
As the systems we are testing on are giving zeroed out memory at new
allocations so far we did not see this test failing.

The purpose of this change is to avoid the cost of supporting many
versions of OpenCV.

It seems LLVM rolled back the clang package the last image was built
with.

In https://github.com/opencv/opencv/pull/26054 a channel count was added
instead of the usual MatType.

Remove WITH_PNG=OFF setting in ci-opencv.sh to avoid a warning-as-error
for an unused function in opencv/modules/core/test/test_arithm.cpp

Change test patterns in ci-opencv.sh to be more specific to avoid
matching some new long-running tests in OpenCV.

Image rebuilt with latest packages

OpenCV 4.10.0 has a warning with latest clang, so the specific warning
is turned off to let the CI pass.

The use of mutable variables across many lambdas had the effect of
action-at-a-distance [1].

[1] https://en.wikipedia.org/wiki/Action_at_a_distance_(computer_programming)

Repurposes existing u8 implementations of remap_s16 for Neon and SVE2,
and makes them work for source type u16.

This is intended to catch potential issues in adapters/opencv.

It is necessary to remove -DOPENCV_DISABLE_THREAD_SUPPORT=ON
otherwise the build will fail with a warning about unused function
getNumberOfCPUsImpl.

Use KLEIDICV_MAXIMUM_CHANNEL_COUNT instead of the magic number 4 in some
test code.

Michael Platings authored Dec 12, 2024 and Denes Tarjan committed Dec 18, 2024

Also update signature of remap and warp_perspective functions to avoid
the same mistake there.