PIA-Group / BioSPPy

Biosignal Processing in Python
Other
569 stars 273 forks source link

eda basic_scr mismatch in returned indices #68

Closed malfarasplux closed 3 years ago

malfarasplux commented 4 years ago

test_biosppy_basic_scr.zip

eda.basic_scr method

The basic_scr seems to be cutting wrongly the returned indices. After detecting null-derivative singular points, onset and peak indices to be returned are arranged to be returned in pairs.

However, the way peaks and onsets are discarded leads to (peak,onset,peak,onset) returns instead of (onset,peak,onset,peak) returns that would be more meaningful.

https://github.com/PIA-Group/BioSPPy/blob/master/biosppy/signals/eda.py

afonsocraposo commented 3 years ago

Fixed with commit https://github.com/PIA-Group/BioSPPy/commit/709c6a121cfc3ea2181eb29f7c89049552cb5802.