search

sccn / EEG-BIDS

21 stars 17 forks source link

Rotated electrodes in ds003645 #201

Closed arnodelorme closed 2 months ago

arnodelorme commented 4 months ago

I have checked that the import is OK

https://github.com/sccn/eeglab/issues/747

Maybe run the export script again @dungscout96

dungscout96 commented 3 months ago

In the original version of the export script, there's seem to be a line to change the nose direction EEG = pop_chanedit(EEG,'nosedir','+Y'); I re-exported the data and re-ran the pipeline. Looking at the processed data, it seems that the command didn't change the nose direction.

I tested with two different set files, one from a different dataset. The field nosedir: '+X' in EEG.chaninfo doesn't change after running the command, with a message Note for expert users: Nose direction is now set from '+Y' to default +X in EEG.chanlocs. Is this the expected behavior?

dungscout96 commented 3 months ago

After discussing with Arnaud this behavior is ok and the nose direction has actually been corrected. The issue might lie somewhere else in the export script or BIDS tool

dungscout96 commented 3 months ago

The issue can be fixed by #208

dungscout96 commented 2 months ago

Duplicate of https://github.com/sccn/NEMAR-pipeline/issues/72