Replace NaNs by 0 for continuous variables at the 1st level (reaction time)

LIMO-EEG-Toolbox / limo_tools

Hierarchical Linear Modelling for MEEG data

https://limo-eeg-toolbox.github.io/limo_meeg/

Other

58 stars 28 forks source link

Replace NaNs by 0 for continuous variables at the 1st level (reaction time) #118

Closed arnodelorme closed 1 year ago

arnodelorme commented 1 year ago

I put it here, so we do not forget. Should we have a checkbox in pop_limo asking users if they want to replace NaNs by 0 if a lot of NaNs are detected for a continuous variable?

CPernet commented 1 year ago

Actually, it was doing it line 163 -- but it would necessarily fail testing singularity det(Cont'*Cont) == 0. I just moved the code ; does it do what you wanted? https://github.com/LIMO-EEG-Toolbox/limo_tools/commit/4048aedf1d7acbd82750110405b24f8ae71ff86e

CPernet commented 1 year ago

follow-up?