• This project
    • Loading...
  • Sign in

ftract_dev / ImaGIN2 · Issues

Go to a project

GitLab

  • Go to group
  • Project
  • Activity
  • Files
  • Commits
  • Pipelines 0
  • Builds 0
  • Graphs
  • Milestones
  • Issues 11
  • Merge Requests 0
  • Members
  • Labels
  • Wiki
  • Forks
  • Network
  • Create a new issue
Closed
Open
Issue #24 opened 2021-02-22 08:51:56 UTC byBlandine Chanteloup-Foret@chantelb

0009MAR - BipolarMontage step - Multiple monopolar channels with the same label found

Edited 2022-02-04 09:00:20 UTC
  • Blandine Chanteloup-Foret
    @chantelb commented 2021-03-09 09:29:27 UTC
    Master

    Fixed by Anthony 22 feb 2021. Commit aedb8927 : Fix how to manage ', p, - in electrode label.

    Il y avait en fait 2 soucis:

    1) Un mélange de channels avec des ' et des p qui étaient mal interprétés. Par exemple, selon la convention F-Tract, A'1 et Ap1 doivent être transformés en Ap01 et AP01, mais au lieu de ça le code générait deux fois Ap01 d'où l'erreur "Multiple monopolar channels with the same label found" durant le calcul du montage bipolaire.

    2) Un autre souci, qui n'a pas généré d'erreurs, provient du fait que certaines channels dans le .csv s'appellent par exemple "T-TH01" avec un tiret ce qui ne respecte pas la convention F-Tract et donc il n'y a pas eu de correspondances avec le fichier SPM dans lequel il va systématiquement manquer les coordonnées pour ces channels. J'ai également corrigé ce souci mais je ne suis pas sûr que ça soit le comportement souhaité.

    Les fix modifient les fichiers SPM du CRF (correction des noms des channels + coordonnées manquantes) et il faut donc faire un Rollback et il faudra refaire le quality control.

    Edited 2021-03-09 09:31:21 UTC
  • Blandine Chanteloup-Foret
    @chantelb commented 2021-03-09 09:32:39 UTC
    Master

    Relance steps "Convert" to "BadChannels" 25 feb 2021 -> Despite RollBack=True, only the new Crop are displayed on the Crop validation web page!

    Edited 2021-03-09 09:33:39 UTC
  • Blandine Chanteloup-Foret
    @chantelb commented 2021-05-17 16:48:10 UTC
    Master

    Fixed by @anthonyboyer in 8883f903

  • Blandine Chanteloup-Foret
    @chantelb 2021-05-17 16:48:23 UTC
    Master

    Reassigned to @chantelb

  • Blandine Chanteloup-Foret
    @chantelb commented 2021-05-21 08:09:36 UTC
    Master

    Re-processing OK from "Convert" to "BadChannels" steps with Rollback=True and .txt/.csv files deletion - 18/05/2021.

    Ready to be validated in the https://f-tract.eu:85/ftqualitycontrol/crop/0009MAR/ webpage.

  • Blandine Chanteloup-Foret
    @chantelb 2021-05-21 08:09:50 UTC
    Master

    Reassigned to @davido

  • Blandine Chanteloup-Foret
    @chantelb commented 2021-05-21 08:45:04 UTC
    Master

    mentioned in issue https://gin11-git.ujf-grenoble.fr/ftract_dev/ft_pipeline/issues/135

  • Blandine Chanteloup-Foret
    @chantelb 2021-05-21 08:45:04 UTC
    Master

    Status changed to closed

  • Blandine Chanteloup-Foret
    @chantelb commented 2022-01-31 11:15:43 UTC
    Master

    Same error for:

    0004MIL, 0007MIL, 0039MIL, 0047MIL, 0077MIL

    To be reprocessed and revalidated...

  • Blandine Chanteloup-Foret
    @chantelb 2022-01-31 11:15:43 UTC
    Master

    Status changed to reopened

  • Blandine Chanteloup-Foret
    @chantelb 2022-01-31 11:15:56 UTC
    Master

    Reassigned to @jedynakm

  • Blandine Chanteloup-Foret
    @chantelb 2022-01-31 11:16:01 UTC
    Master

    Reassigned to @chantelb

  • Blandine Chanteloup-Foret
    @chantelb commented 2022-02-04 09:00:07 UTC
    Master

    0004MIL - step Electrodes - error : Duplicated label found: c3

  • Blandine Chanteloup-Foret
    @chantelb 2022-02-04 09:00:20 UTC
    Master

    Reassigned to @anthonyboyer

  • Please register or login to post a comment
24 of 41
Prev Next
Anthony Boyer
Assignee
Anthony Boyer @anthonyboyer
Assign to
None
Milestone
None
Assign milestone
None
Due date
None
4
4 participants
Reference: ftract_dev/ImaGIN2#24