Conversation
| usecols=retained_columns | ||
| ).to_numpy() | ||
|
|
||
| if len(line_splitted) < CYTON_TOTAL_NB_CHANNELS: |
There was a problem hiding this comment.
Cela permettait de voir s'il y avait un problème dans le fichier envoyé. Par exemple, si le Cyton se ferme et se rallume momentanément, il y a aura deux commentaires qui vont indiquer le début de l'enregistrement. Voir docu %STOP AT et %START AT
There was a problem hiding this comment.
Peut-être juste mettre un try catch autour read_csv pour les lignes de commentaire où le retained_columns ne seront pas présentes. On pourra renvoyer une erreur 400 avec l'explication dans le body.
There was a problem hiding this comment.
Est-ce qu'on drop les lignes qui pètent sinon? pandas le fait bien
There was a problem hiding this comment.
idk, s'il manque des samples le fichier ne vaut pas probablement pas grand chose
There was a problem hiding this comment.
Les lignes tu veux dire? Oui, c'est un peu embêtant. Dans le cas où il n'y a qu'un arrêt de ~ 30 secondes, on peut drop ces lignes. Dans le cas d'un arrêt de > 5 minutes, ça ne marcherait pas tant de juste drop les lignes, comme il y a eu un arrêt non négligeable. On ne supporte pas non plus une séquence de nuit non contigüe, ni dans la classification et ni dans les visualisations. C'est pourquoi je pensais refuser le fichier dans ce cas.
De toute façon, c'est un cas limite, on peut pour l'instant drop les mauvaises lignes.
There was a problem hiding this comment.
Ok, j'ai catch l'erreur de pandas et raise ClassificationError
3 participants
No description provided.