Skip to content

Add signal module with signal resampling function#3613

Merged
opencv-pushbot merged 1 commit intoopencv:4.xfrom
amishutin:signal_module
Jan 12, 2024
Merged

Add signal module with signal resampling function#3613
opencv-pushbot merged 1 commit intoopencv:4.xfrom
amishutin:signal_module

Conversation

@amishutin
Copy link
Copy Markdown
Contributor

@amishutin amishutin commented Dec 28, 2023 •
edited by opencv-alalek
Loading

Added signal module with resampling function implemented with cubic interpolation function and a filtering function based on Kaiser window and Bessel function, used to construct a FIR filter.
Detail: https://en.wikipedia.org/wiki/Sample-rate_conversion

There are provided:

  • signal resampling API
  • optimization done with universal intrinsics
  • functional tests with reference signals
  • performance tests

Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
  • The PR is proposed to the proper branch
  • There is a reference to the original bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake
force_builders=Linux x64 Debug,Win32,ARMv7,ARMv8

@amishutin amishutin changed the title Add signal module with signal resampling function Add signal module with signal resampling function Dec 28, 2023
@amishutin amishutin changed the title Add signal module with signal resampling function Add signal module with signal resampling function Dec 28, 2023
@amishutin amishutin force-pushed the signal_module branch 4 times, most recently from 529db0a to ec7edb9 Compare January 9, 2024 06:35
@amishutin amishutin closed this Jan 9, 2024
@amishutin amishutin reopened this Jan 9, 2024
opencv-alalek
opencv-alalek reviewed Jan 9, 2024
View reviewed changes
Copy link
Copy Markdown

@opencv-alalek opencv-alalek left a comment

Choose a reason for hiding this comment

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

Thank you for the contribution!

Please take a look on comments below.

modules/signal/test/test_signal.cpp Outdated

float MSE(const Mat1f &outSignal, const Mat1f &refSignal)
{
auto mse{0.f};
Copy link
Copy Markdown

@opencv-alalek opencv-alalek Jan 9, 2024

Choose a reason for hiding this comment

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

Please use

float mse = 0.f;

here and below (instead of "auto" scalars).

To workaround old ARMv7 compiler.

modules/signal/src/signal_resample.cpp Outdated
static float Bessel(float x)
{
int k = 12; // approximation parameter
float defmul = powf(x, 2) * 0.25f;
Copy link
Copy Markdown

@opencv-alalek opencv-alalek Jan 9, 2024

Choose a reason for hiding this comment

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

powf(x, 2)

x * x should be faster.

The same is below.

modules/signal/src/signal_resample.cpp Outdated
{
tabs[i] = 2 * fc * (i - (ntabs - 1) / 2);
}
float *tmparr = new float[ntabs];
Copy link
Copy Markdown

@opencv-alalek opencv-alalek Jan 9, 2024

Choose a reason for hiding this comment

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

Please use cv::AutoBuffer or std::vector instead. Avoid manual memory management.

modules/signal/src/signal_resample.cpp Outdated
for (int i = 0; i < ntabs; ++i)
{
tabs[i] = std::sin(tmparr[i]) / tmparr[i];
tabs[i] *= Bessel(beta * sqrtf((float)1 - powf((2 * i / (float)(ntabs - 1) - 1), 2))) / Bessel(beta);
Copy link
Copy Markdown

@opencv-alalek opencv-alalek Jan 9, 2024

Choose a reason for hiding this comment

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

/ (float)(ntabs - 1)

could be moved out of the loop as pre-computed 2.0f / (float)(ntabs - 1)

modules/signal/src/signal_resample.cpp Outdated

/////////////// cubic Hermite spline (OpenCV's Universal Intrinsics) ///////////////
#if (CV_SIMD || CV_SIMD_SCALABLE)
static v_float32 simd_cubicHermite(v_float32 v_A, v_float32 v_B, v_float32 v_C, v_float32 v_D, v_float32 v_t)
Copy link
Copy Markdown

@opencv-alalek opencv-alalek Jan 9, 2024

Choose a reason for hiding this comment

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

v_float32

  1. static inline
  2. const v_float32& to workaround alignment issues (see Win32 builder)

@amishutin amishutin force-pushed the signal_module branch 3 times, most recently from c82e7f5 to 71d52c8 Compare January 10, 2024 11:28
@opencv-alalek
Copy link
Copy Markdown

opencv-alalek commented Jan 10, 2024

Thank you for the updates!
Could you please try to run signal tests in Debug mode to verify failed assertions?

@amishutin
Copy link
Copy Markdown
Contributor Author

amishutin commented Jan 11, 2024

Thank you for the updates! Could you please try to run signal tests in Debug mode to verify failed assertions?

Found the issue in scalar code. Fixed

opencv-alalek
opencv-alalek reviewed Jan 11, 2024
View reviewed changes
Copy link
Copy Markdown

@opencv-alalek opencv-alalek left a comment

Choose a reason for hiding this comment

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

Thank you for update!

modules/signal/include/opencv2/signal.hpp
// of this distribution and at http://opencv.org/license.html
#ifndef OPENCV_SIGNAL_HPP
#define OPENCV_SIGNAL_HPP

Copy link
Copy Markdown

@opencv-alalek opencv-alalek Jan 11, 2024

Choose a reason for hiding this comment

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

Please add module declaration for documentation: http://pullrequest.opencv.org/buildbot/export/pr_contrib/3613/docs/

/**
@defgroup signal Signal Processing

This module includes signal processing algorithms.

@}
*/

Copy link
Copy Markdown
Contributor Author

@amishutin amishutin Jan 11, 2024

Choose a reason for hiding this comment

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

Added declaration

opencv-alalek
opencv-alalek approved these changes Jan 11, 2024
View reviewed changes
Copy link
Copy Markdown

@opencv-alalek opencv-alalek left a comment

Choose a reason for hiding this comment

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

Well done 👍

@amishutin
Copy link
Copy Markdown
Contributor Author

amishutin commented Jan 11, 2024

Added fix for Windows10-x64 warnings check

@opencv-alalek opencv-alalek self-assigned this Jan 12, 2024
@opencv-pushbot opencv-pushbot merged commit 674cec1 into opencv:4.x Jan 12, 2024
@opencv-alalek opencv-alalek mentioned this pull request Jan 15, 2024
Merged
@asmorkalov asmorkalov mentioned this pull request Jan 23, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mshabunin mshabunin mshabunin approved these changes

@opencv-alalek opencv-alalek opencv-alalek approved these changes

Assignees

@opencv-alalek opencv-alalek

Labels

category: signal feature new module

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@amishutin @opencv-alalek @mshabunin @opencv-pushbot