Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ds003645/sub006 rotation issues (Scott) #72

Open
arnodelorme opened this issue Mar 25, 2024 · 13 comments
Open

ds003645/sub006 rotation issues (Scott) #72

arnodelorme opened this issue Mar 25, 2024 · 13 comments
Assignees

Comments

@arnodelorme
Copy link
Collaborator

  • one clearly bad channel got through the cleaning
  • the maps are all turned 90 deg to the left
  • the browser option does not work
    whole dataset:
  • fix the montage orientation
  • make the browser run on these data - ask Yahya how to do this
  • check and adjust the data cleaning parameters (see above)
  • please run the pipeline again using the new AMICA call Jason provided
  • although Dung told me that the runs are concatenated before running ICA decomposition
    in this dataset that is not the case - each run has a separate decomposition.
    Please concatenate the runs and run a single decomposition -- as you thought had been done already.
  • when you run the pipeline, please add the PMI (pairwise mutual information) and save the output and the component subspace plots for review.
@dungscout96
Copy link
Collaborator

  • Which cleaning parameters should I use?
  • @neuromechanist Can you point me to the updated AMICA code? Also I don't remember if the PMI scripts have been fixed per our last discussion?
  • @arnodelorme We discussed last time that it might not be necessary to concatenate the runs. Should we discuss this again?

@neuromechanist
Copy link
Member

Hi Dung,

You can use sccn/amica/runamica17_nsg.m to run efficiently on Expanse. You should use [batch, 1] in the input so that the job is submitted as a job. Other parameters seem to be ok and would not need any change.

Let me know how it goes.

P.S.: It seems that we are once again at the end of NSG allocation. I hope that you can submit new tasks.

@arnodelorme
Copy link
Collaborator Author

arnodelorme commented Apr 8, 2024

Too short for ICA would be if the number of samples is less than nchannel^2*20 samples.

@neuromechanist
Copy link
Member

BTW, the amica bag code is available from:

sccn/amica/runamica17_nsg.m

@dungscout96
Copy link
Collaborator

@neuromechanist the AMICA call ran yet seemed to exit with error. There's no output in the amicaout directory (see screenshot) and there's no message to indicate that the process finished in the log file (attached)
image
amica.30062187.txt

@neuromechanist
Copy link
Member

Interesting, pinging @japalmer29 for potential insights about the error:
forrtl: severe (66): output statement overflows record

@japalmer29
Copy link

@dungscout96 Could you post the matlab command you used to run it?

@dungscout96
Copy link
Collaborator

@japalmer29
Copy link

@dungscout96 could you post the runamica17_nsg(...) command with all the arguments?

@dungscout96
Copy link
Collaborator

The exact command was runamica17_nsg(EEG, 'batch', 1)

@neuromechanist
Copy link
Member

It seems that the problem is that runamica17_nsg does not return anything in batch mode. I just added a mirror return (EEG=dat) in sccn/amica@3257aba, so it returns dat (which, most of the case, is the EEG structure) in the batch mode as well.

However, to load data later, you should use EEG = eeg_loadamica(EEG, outdir, 1) for single model data.

@arnodelorme arnodelorme reopened this Aug 15, 2024
@arnodelorme arnodelorme changed the title ds003645/sub006 issues (Scott) ds003645/sub006 rotation issues (Scott) Aug 15, 2024
@dungscout96
Copy link
Collaborator

EEG=pop_chanedit(EEG, 'forcelocs',[],'transform',{'TMP = X; X = Y; Y = -TMP;'});

@arnodelorme
Copy link
Collaborator Author

Add HED to https://nemar.org/dataexplorer/detail?dataset_id=ds002718

Add new columns to NEMAR.json to disable ICA for that dataset

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants