psambit9791 / jdsp

A Java Library for Digital Signal Processing
https://jdsp.dev
MIT License
240 stars 45 forks source link

Implement Unit testing for long signals in STFT and ISTFT #29

Closed SiboVG closed 2 years ago

SiboVG commented 2 years ago

This commit implements testing for a long-duration signal for STFT and ISTFT, to check whether the new FFT-implementation works

codecov[bot] commented 2 years ago

Codecov Report

Merging #29 (30178a6) into master (fc8b8f3) will decrease coverage by 0.19%. The diff coverage is 71.23%.

:exclamation: Current head 30178a6 differs from pull request most recent head 7da9f83. Consider uploading reports for the commit 7da9f83 to get more accurate results Impacted file tree graph

@@             Coverage Diff              @@
##             master      #29      +/-   ##
============================================
- Coverage     83.40%   83.21%   -0.20%     
- Complexity     1022     1057      +35     
============================================
  Files            61       62       +1     
  Lines          3743     3879     +136     
  Branches        679      703      +24     
============================================
+ Hits           3122     3228     +106     
- Misses          419      437      +18     
- Partials        202      214      +12     
Impacted Files Coverage Δ
.../github/psambit9791/jdsp/filter/adaptive/NLMS.java 66.66% <66.66%> (ø)
...m/github/psambit9791/jdsp/filter/adaptive/LMS.java 69.35% <69.35%> (ø)
...ub/psambit9791/jdsp/transform/DiscreteFourier.java 95.74% <100.00%> (+0.09%) :arrow_up:
...github/psambit9791/jdsp/transform/FastFourier.java 50.00% <100.00%> (+9.18%) :arrow_up:
...com/github/psambit9791/jdsp/transform/Hilbert.java 78.68% <100.00%> (+4.55%) :arrow_up:
...it9791/jdsp/transform/InverseShortTimeFourier.java 70.83% <100.00%> (+3.22%) :arrow_up:
...b/psambit9791/jdsp/transform/ShortTimeFourier.java 73.33% <100.00%> (+2.22%) :arrow_up:
...psambit9791/jdsp/transform/InverseFastFourier.java 49.18% <0.00%> (+3.27%) :arrow_up:
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update fc8b8f3...7da9f83. Read the comment docs.