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

Function naming conflict and ICLabel #509

Closed
arnodelorme opened this issue Jul 23, 2022 · 3 comments
Closed

Function naming conflict and ICLabel #509

arnodelorme opened this issue Jul 23, 2022 · 3 comments

Comments

@arnodelorme
Copy link
Collaborator

One weird issue we've been having on her end is that ICLabel produces NaN for all class probabilities when ICLabel is run in sequence with the various processing steps (filtering/ASR/AMICA). However, when MATLAB is restarted, ICLabel works normally. Only when run in sequence with the various preprocessing steps does it not work.

@arnodelorme
Copy link
Collaborator Author

Fieldtrip bug submitted fieldtrip/fieldtrip#2080

@jjurgiel
Copy link

jjurgiel commented Jul 24, 2022

Full details:

Recreated this issue using Matlab 2020b, EEGLAB 2021.1, Fieldtrip-lite20210601, EEGLAB prompted me to install Fieldtrip-lite20220714 when running the pop_multifit function (below), which I hadn't seen happen before.

[~,coordinateTransformParameters] = coregister(EEG.chanlocs, ...
        'D:\Documents\eeglab2021.1\plugins\dipfit\standard_BEM\elec\standard_1005.elc', 'warp', 'auto', 'manual', 'off');
    
% Dipfit settings
EEG = pop_dipfit_settings( EEG, ...
    'hdmfile', 'D:\Documents\eeglab2021.1\plugins\dipfit\standard_BEM\standard_vol.mat', ...
    'mrifile', 'D:\Documents\eeglab2021.1\plugins\dipfit\standard_BEM\standard_mri.mat',...
    'chanfile', 'D:\Documents\eeglab2021.1\plugins\dipfit\standard_BEM\elec\standard_1005.elc',...
    'coordformat', 'MNI', 'coord_transform', coordinateTransformParameters,'chansel', 1:EEG.nbchan);

EEG = pop_multifit(EEG, 1:size(EEG.icaact,1),'threshold', 100);

EEG = iclabel(EEG);

The pop_multifit in particular added a number of /external/ Fieldtrip folders to the matlab path. Running ICLabel gives NaN values, unless you remove /external/signal from the path prior to running ICLabel.

rmpath('D:\Documents\eeglab2021.1\plugins\Fieldtrip-lite20220714\external\signal')

@arnodelorme
Copy link
Collaborator Author

This has been fixed by removing external folders in Fieldtrip by default.

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

2 participants