OverLordGoldDragon / ssqueezepy

Synchrosqueezing, wavelet transforms, and time-frequency analysis in Python
MIT License
618 stars 96 forks source link

Change default `gamma` #45

Closed OverLordGoldDragon closed 2 years ago

OverLordGoldDragon commented 2 years ago

Existing default is overly aggressive and may filter legitimate signals; changed to a more reasonable value.

Recommended to avoid passing a small-valued input (try standardizing or otherwise rescaling such that x.max() >= 1), especially if the signal obeys a power scaling law (e.g. ~ 1/f with EEG/MEG, and similar in audio).

codecov-commenter commented 2 years ago

Codecov Report

Merging #45 (d0ce51e) into master (b08fcb8) will not change coverage. The diff coverage is 75.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #45   +/-   ##
=======================================
  Coverage   92.38%   92.38%           
=======================================
  Files          17       17           
  Lines        3428     3428           
=======================================
  Hits         3167     3167           
  Misses        261      261           
Impacted Files Coverage Δ
ssqueezepy/_ssq_stft.py 87.17% <50.00%> (ø)
ssqueezepy/_ssq_cwt.py 92.45% <100.00%> (ø)

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 b08fcb8...d0ce51e. Read the comment docs.