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

Resting state with dataset and example #400

Merged
merged 59 commits into from Jun 22, 2023

Conversation

gcattan
Copy link
Contributor

@gcattan gcattan commented Jun 14, 2023

This PR introduces a resting state paradigm and a new dataset. An example is provided.
The code is adapted from @plcrodrigues's repository: https://github.com/plcrodrigues/py.PHMDML.EEG.2017-GIPSA

@bruAristimunha @sylvchev FYI

Gregoire Cattan and others added 30 commits May 31, 2023 13:56
fix: inversion 12 blocks of 5 repetitions
Co-authored-by: Sylvain Chevallier <sylvain.chevallier@universite-paris-saclay.fr>
- add comments
@bruAristimunha bruAristimunha self-requested a review June 19, 2023 20:08
Comment on lines +171 to +179
if "Target" in event_id and "NonTarget" in event_id:
if (
type(event_id["Target"]) is list
and type(event_id["NonTarget"]) == list
):
event_id_new = dict(Target=1, NonTarget=0)
events = mne.merge_events(events, event_id["Target"], 1)
events = mne.merge_events(events, event_id["NonTarget"], 0)
event_id = event_id_new
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Sara04, is this related to your issue with the p300?

Copy link
Collaborator

@bruAristimunha bruAristimunha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gcattan, I must say, your code looks really cool!

I have a few questions rather than requests for changes.

First, I would like you to create some testing for the new paradigm. Additionally, could you modify the "dataset_summary" file in the documentation to incorporate information about the new paradigm?

As we plan to expand this dataset category, it would be beneficial to include some introductory text about the new paradigm in this file, maybe a figure if you have it easy. This way, we can attract new users to utilize it.

@gcattan
Copy link
Contributor Author

gcattan commented Jun 21, 2023

Thanks, @bruAristimunha. I put here some more comments and additional testing. I do not have a figure ready at the moment, unfortunately.

@bruAristimunha
Copy link
Collaborator

Firstly, I want to express my gratitude to both @gcattan and @plcrodrigues for introducing this new paradigm. I have reviewed the tutorial and made a few minor adjustments. These changes primarily focus on improving the formatting and markdown style of the tutorial. I hope you don't mind these final modifications 🙏🏽. Once everything is green, I'll apply the merge =)

@gcattan
Copy link
Contributor Author

gcattan commented Jun 22, 2023

All good to me :) thank you for the last changes :)

@bruAristimunha bruAristimunha merged commit 2821954 into NeuroTechX:develop Jun 22, 2023
8 checks passed
@bruAristimunha
Copy link
Collaborator

Hey @dengemann,

I was thinking, and I think you might like to know that moabb has resting stage dataset,

I hope that in the future we can put more public dataset in this category, maybe on your side :)

@plcrodrigues
Copy link

plcrodrigues commented Jun 29, 2023

I'm not sure if this is well known, but in the PhysionetMI there's also a resting state part.

They are the runs 1 and 2 as explained here. This might be useful for the moabb package.

I won't have time to do much about it now, but I will leave this low hanging fruit here :-)

@gcattan
Copy link
Contributor Author

gcattan commented Jun 29, 2023

Sounds like a good use case for the GoShoppingDataset ^^
There is also the alphawaves dataset and this one:
https://archive.ics.uci.edu/dataset/264/eeg+eye+state

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 this pull request may close these issues.

None yet

4 participants