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

NaN value in SCR_Recovery_Index #73

Open
chongluyao opened this issue Jul 19, 2018 · 1 comment
Open

NaN value in SCR_Recovery_Index #73

chongluyao opened this issue Jul 19, 2018 · 1 comment
Labels

Comments

@chongluyao
Copy link

Hi, I'm using the processed_eda function to extract feature from the raw eda in my research. However, I found that there were some NaNs in the "processed_eda['EDA']['SCR_Recovery_Indexes']". Why these NaNs appear? And how should I deal with these NaNs to get the final feature such as mean of recovery times in your suggestion?

@DominiqueMakowski
Copy link
Member

@chongluyao It seems that currently, Recovery is having NA if there is no peak onset in the selected SCR window. However, by looking at it superficially, I'm not sure this is needed. It could potentially be refactored to something a little bit better. I'll try to have a look at it when I'll have some time.

This does not prevent you from computing the means though, sometimes it's better to compute a feature based on few, but clean observations than from noisy and artefacted data.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants