Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

添加数字信号处理工具库(头文件) dsp.hpp #83

Closed
1 of 3 tasks
zhaoxi-scut opened this issue Apr 26, 2024 · 1 comment · Fixed by #155
Closed
1 of 3 tasks

添加数字信号处理工具库(头文件) dsp.hpp #83

zhaoxi-scut opened this issue Apr 26, 2024 · 1 comment · Fixed by #155
Labels
feature 新特性、新功能 wontfix 无意义的 issue 或 pr
Milestone

Comments

@zhaoxi-scut
Copy link
Member

zhaoxi-scut commented Apr 26, 2024

描述功能特点和出发点

初步计划添加有关

  • 离散傅里叶变换
  • 快速傅里叶变换
  • 数字滤波器

的功能

附加内容

@zhaoxi-scut zhaoxi-scut added the feature 新特性、新功能 label Apr 26, 2024
@zhaoxi-scut zhaoxi-scut added this to the 1.3.0 milestone Apr 26, 2024
@zhaoxi-scut zhaoxi-scut changed the title 添加信号处理工具库(头文件) dsp.hpp 添加数字信号处理工具库(头文件) dsp.hpp Apr 26, 2024
@zhaoxi-scut zhaoxi-scut added the wontfix 无意义的 issue 或 pr label Apr 30, 2024
@zhaoxi-scut zhaoxi-scut removed this from the 1.3.0 milestone Apr 30, 2024
@zhaoxi-scut
Copy link
Member Author

zhaoxi-scut commented Apr 30, 2024

很遗憾,本来写的差不多了,做了个基准测试,没想到差距这么大

--------------------------------------------------------------------------------
Benchmark                                      Time             CPU   Iterations
--------------------------------------------------------------------------------
DFT (N=1024) -   RMVL/iterations:10     22746236 ns     22717058 ns           10
FFT (N=1024) -   RMVL/iterations:10       180989 ns       181006 ns           10
DFT (N=1024) - OpenCV/iterations:10        14091 ns        14108 ns           10

加上 OpenCV 也提供了 cv::filter2D 的数字滤波器,因此该增加新功能的 issue 将被暂时关闭,若后期 OpenCV 成为 RMVL 不必要的依赖项时,则继续打开该 issue,并在支持 OpenCV 的情况下启用 cv::dft,否则将采用默认实现,但届时 rm::dft 将删除,rm::fft 将更名为 rm::dft

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 新特性、新功能 wontfix 无意义的 issue 或 pr
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant