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

Error when loading Lee2019_SSVEP dataset #295

Closed
AlexandreBleuze opened this issue May 30, 2022 · 3 comments · Fixed by #298
Closed

Error when loading Lee2019_SSVEP dataset #295

AlexandreBleuze opened this issue May 30, 2022 · 3 comments · Fixed by #298

Comments

@AlexandreBleuze
Copy link

Hi,

I'm currently working on SSVEP paradigm and trying to load the Lee2019_SSVEP dataset. However when loading it with SSVEP filter-bank paradigm, I'm getting the following error "RuntimeError: events cannot be set directly."
I think the error is comming from a manipulation in MNE that is not allowed, from what I searched but I can't get passed it.
My versions are :
Python 3.9.10
Moabb 0.4.6
MNE 1.0.3

A simple code to give the error would be :

from moabb.datasets import Lee2019_SSVEP
from moabb.paradigms import FilterBankSSVEP

# Does not work when settings events and filters
moabb_paradigm = FilterBankSSVEP(n_classes=2, events=["8.57", "12.0"], filters=[[8.1, 9.1], [11.5, 12.5]])

# Does not work much when just setting a number of classes
# moabb_paradigm = FilterBankSSVEP(n_classes=2)

dataset = Lee2019_SSVEP()
Xt, yt, _ = moabb_paradigm.get_data(dataset=dataset, subjects=[1])

I also join a screen of the error I'm getting if you are interested.
Erreur Lee2019_SSVEP

@sylvchev
Copy link
Member

sylvchev commented Jun 1, 2022

I could reproduce the error with newer version of MNE (> 1.0.0). With older MNE versions (like 0.24.1), both settings events+filters or n_classes are working, I obtain Xt signal of shape (100, 62, 4001, 2) and (100, 62, 4001, 2)

@AlexandreBleuze
Copy link
Author

I'll try to use an older version of MNE then. Do you think it is possible to solve this problem for the newest version of MNE ?

@sylvchev
Copy link
Member

sylvchev commented Jun 8, 2022

Yes, I'll fix it in an upcoming PR.

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

Successfully merging a pull request may close this issue.

2 participants