Skip to content

include EEG example dataset#18

Merged
chrisgorgo merged 4 commits intobids-standard:bep006_eegfrom
robertoostenveld:master
Jul 9, 2017
Merged

include EEG example dataset#18
chrisgorgo merged 4 commits intobids-standard:bep006_eegfrom
robertoostenveld:master

Conversation

@robertoostenveld
Copy link
Copy Markdown
Collaborator

This is an example dataset prepared by @arnodelorme, following discussion with @CPernet and me. There is still work to be done on this example to align it better with the EEG-BIDS proposal and the other BIDS standards.

@chrisgorgo
Copy link
Copy Markdown
Contributor

Thanks for putting this together. I can either merge this into the eeg branch or keep it open until EEG-BIDS is ready. What would you prefer?

@chrisgorgo
Copy link
Copy Markdown
Contributor

chrisgorgo commented Jul 7, 2017

BTW event files should be _events.tsv not _event.tsv.

@robertoostenveld
Copy link
Copy Markdown
Collaborator Author

I prefer to pull it into a branch rather than keeping it open. I think it is (conceptually) easier for others to pull the branch from INCF than from my repo. And I do want/hope that others contribute.

I suggest renaming the EEG branch to BEP006. That gives a direction for handling future BEPs on the github repository. Could you also make a BEP00x branch for MEG (I don't know the number)?

@chrisgorgo chrisgorgo changed the base branch from master to bep006_eeg July 9, 2017 15:38
@chrisgorgo
Copy link
Copy Markdown
Contributor

Ok I'm merging this into bep006_eeg. I'll also give you write permission to the repo, but please use PRs to put in changes (it's easier to track progress this way).

I also created bep008_meg.

@chrisgorgo chrisgorgo merged commit 4cfe2e0 into bids-standard:bep006_eeg Jul 9, 2017
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.

2 participants