LIMO-EEG-Toolbox / limo_tools

Hierarchical Linear Modelling for MEEG data
https://limo-eeg-toolbox.github.io/limo_meeg/
Other
59 stars 27 forks source link

CL and CL_list not defined in limo_ecluster_test.m #12

Closed cathymanning closed 6 years ago

cathymanning commented 6 years ago

'CL' and 'CL_list' are not defined in limo_ecluster_test.m so errors are returned when function tries to output pval.

CPernet commented 6 years ago

can you give me the context in which you use the function - it is called by limo_clustering but i cannot see any output errors from running from the GUI ??

thx

cathymanning commented 6 years ago

Hi

I have been using it as a standalone function in my MATLAB script after using limo_ecluster_make.m. I have been in touch with Guillaume who suggested using these functions in this way and said I should post a bug report - let me know if I can give any more information.

Best wishes

Cathy


From: Cyril Pernet notifications@github.com Sent: 03 April 2018 16:41:31 To: LIMO-EEG-Toolbox/limo_eeg Cc: cathymanning; Author Subject: Re: [LIMO-EEG-Toolbox/limo_eeg] CL and CL_list not defined in limo_ecluster_test.m (#12)

can you give me the context in which you use the function - it is called by limo_clusteringhttps://github.com/LIMO-EEG-Toolbox/limo_eeg/blob/master/limo_clustering.m but i cannot see any output errors from running from the GUI ??

thx

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/LIMO-EEG-Toolbox/limo_eeg/issues/12#issuecomment-378295941, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Afgc11B2hrGMtN3FwuY0xO4LqDukXyKGks5tk5grgaJpZM4TE0rl.

CPernet commented 6 years ago

If you call limo_clustering it should perform the all thing fine - that the way we do it within the toolbox. If you put some data and the bit of script / line of code somewhere I can access, I'm happy to look into it though.

CPernet commented 6 years ago

now pushed to master - fixed