Closed nescirem closed 5 years ago
Merging #46 into master will increase coverage by
0.88%
. The diff coverage is100%
.
@@ Coverage Diff @@
## master #46 +/- ##
==========================================
+ Coverage 68.5% 69.39% +0.88%
==========================================
Files 17 17
Lines 1810 1820 +10
Branches 166 167 +1
==========================================
+ Hits 1240 1263 +23
+ Misses 538 528 -10
+ Partials 32 29 -3
Impacted Files | Coverage Δ | |
---|---|---|
PyEMD/tests/test_ceemdan.py | 97.89% <100%> (-0.95%) |
:arrow_down: |
PyEMD/CEEMDAN.py | 94.3% <100%> (+4.96%) |
:arrow_up: |
PyEMD/EMD.py | 87.5% <0%> (+0.24%) |
:arrow_up: |
PyEMD/tests/test_eemd.py | 100% <0%> (+3.17%) |
:arrow_up: |
PyEMD/EEMD.py | 83.58% <0%> (+7.46%) |
:arrow_up: |
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 de373ec...09993ee. Read the comment docs.
S
should not be changed. So I useS[:] = S/scale_s
instead ofS = S/scale_s
to fix it.ceemdan
multiple times in one main program, the program will only use the random noise which was generated in the first call, though the different random seed is given before each call. So I cleaned the relevant list/array after each call to the function. Maybe there will be a better way to fix that? I need some help.I understand signal processing is not something that can be sloppy, so I test it to check I have Fixed that: (IMFs' MD5 is calculated to make sure the IMFs in a different case is same or not)
Before: (seed=114514, MD5=f5ac70aa2e917d87187d8b76207354e0) Fig.1 case A.1 The original Signal is scaled and the residuum is wrong.
After: (seed=114514, MD5=f5ac70aa2e917d87187d8b76207354e0) Fig.2 case A.2
Before: (seed=89889, MD5=0b528aef0f9c710ef02a66685dd4e8ea) Fig.3 case B.1
After: (seed=89889, MD5=0b528aef0f9c710ef02a66685dd4e8ea) Fig.4 case B.2
After: (random seed, MD5=9452d9dd49ba9f16a7eb18d60519bf31) Fig.5 case C
After: (another random seed, MD5=cc2aa1a1d229899e64f59a88090ead5e) Fig.6 case D
Obviously what I did does not change the IMFs (if the random seed is the same).
Sample code of the case A.2 is here: