search

lkorczowski / Tinnitus-n-Sleep

Detecting events in sleeping tinnitus patients
MIT License
1 stars 0 forks source link

Refactoring (Second PR) #5

Closed lkorczowski closed 4 years ago

lkorczowski commented 4 years ago

This PR is a bit more complicated to review because it is mainly on the notebook. Please review what you can do.

RobinGuillard commented 4 years ago

J'ai pu revoir le notebook. Tout d'abord, les imports depuis tinnsleep ne pouvait pas être fait aisément, aussi, j'ai copié en dur les fonctions de config et de data dans le notebook (je sais que ça ne va pas te plaire, mais j'ai pas trop le choix pour que ça marche de mon côté, je pense que ça marche pour toi car tu fais tourner sur pycharm j'imagine...) Ensuite, j'ai eu une erreur dans la cellule "Ouvre le fichier sur lequel on veut travailler": en effet, ça coinçait sur la fonction raw.filter et la raison était assez obscure (sur des picks qui étaient invalides). J'ai pas totalement compris le pourquoi et le comment, mais j'ai l'intuition que le problème vient du fait que les channels "1", "2", et "F" n'existent pas dans le montage 1020 classique. Aussi, afin de leurrer le système pour que ça marche j'ai remarqué que si on labélisait les channels comme EEG plutôt qu'EMG ça marchait. Donc j'ai fait cette modif. Un easy fix, qui fait que nos channels sont labélisées comme eeg alors que c'est de l'EMG, mais bon je pense que si on le sait ça ne va pas nous faire de gros problèmes...