methlabUZH / automagic

Automagic
GNU General Public License v3.0
95 stars 32 forks source link

bug: cannot merge subject, missing field in self.params #15

Closed behinger closed 4 years ago

behinger commented 5 years ago

Hi! (developer version) I'm not sure why this error occured. "Block.m" in Line 223 is throwing an error because

K>> autParams.Settings

ans = 

  struct with fields:

    trackAllSteps: 1
      pathToSteps: 'C:\Users\behinger\cloud\PostDoc\exercises\readinessPotential\bids_r7_results\sub-201\eeg\/allSteps_sub-201_eeg'

K>> self.params.Settings

ans = 

  struct with fields:

    trackAllSteps: 1

Are not equal (for obvious reasons, one field is missing). Not sure if this is a bug, or if I messed up something along the way. It could also be that selecting the "save all steps on the way" option raises this error somehow.

All other fieldnames(autParams) are identical

nickilanger commented 5 years ago

Hi Benedikt,

Can you extend your description, what exactly you did? Did you try to run the preprocessing through the command line? Can you provide the code you used?

Best,

Nicolas

behinger commented 5 years ago

Hi! I actually used the gui :-) grafik grafik

Pretty straight forward analysis pipeline Best, Bene

nickilanger commented 5 years ago

This looks straight forward indeed. :) If you send me one of your files and the channel locations file, I can test it for you.

Best,

Nicolas

behinger commented 5 years ago

(sorry I missed the notification). I analysed this dataset: https://osf.io/s3fdv/ subject 201 with this chanloc file: BioSemi64.loc.txt

nickilanger commented 5 years ago

Hi Benedikt,

Thanks for the files. I'm running the preprocessing on my computer now. I will let you know, how it goes. I've noticed two things:

  1. The .bdf file itself has the channel location already included (if you import it with EEGlab). But I added the channel location file anyway in Automagic.

  2. Based on your settings (print screens) you excluded channel index 80, but there are only 79 channels. You have to provide the channel index not the channel name. So, this might be the error you encountered.

Best,

Nicolas

behinger commented 5 years ago

hi! thanks for your help!

Best, Bene

nickilanger commented 5 years ago

Hi Bene,

this is good idea. We will implement this.

did you try to BIOSIG toolbox within EEGlab? that's how I did it.

I also ran the preprocessing with your data, you shared and IC Label ran after some problems with the MEX files:

error_ICLabel

The preprocessed file looks actually quiet good. NPO_S201

nickilanger commented 5 years ago

Could automagic throw an error if you try to exclude channels that are not there?

Done! it's implemented in the development branch.