Skip to content

ds000117 EEGonly added#66

Closed
CPernet wants to merge 30 commits intobids-standard:bep006_eegfrom
CPernet:master
Closed

ds000117 EEGonly added#66
CPernet wants to merge 30 commits intobids-standard:bep006_eegfrom
CPernet:master

Conversation

@CPernet
Copy link
Copy Markdown
Contributor

@CPernet CPernet commented Mar 21, 2018

This is the full data set for EEG only using the latest BIDS spec

suyashdb and others added 29 commits May 31, 2017 22:18
added missing participants file
added sub-08 to ds008/participants.tsv
…and an incorrect first column containing event number (should be onset)

bids-validator returned First column of the events file must be named 'onset' (code: 20 - EVENTS_COLUMN_ONSET)
FIX - fixed events.tsv, which had an empty first header line element …
added pet_phno and hcp_example_bids directories
Multiecho bold example as per BIDS 1.0.2
 ENH: Copy physio data for DS210 from OpenFMRI
should be put back when PET BEP is merged

This reverts commit 61df79b.
from the multimodal Face dataset, we extracted EEG channels from the MEG
- stored here as .set
@chrisgorgo
Copy link
Copy Markdown
Contributor

It seem this PR is mixing commits from multiple sources.

Additionally ds000117-EEGonly datasets seems to use subXXX instead of sub-XXX. Stimuli files should go into /stimuli (lower caps).

@chrisgorgo
Copy link
Copy Markdown
Contributor

I'm still seeing a lot of changes that seems unrelated (for example deletion of a lot of files from 7t_trt). Not sure what happened here. Maybe it would be best to move the changes manually and start over from bep006_eeg.

robertoostenveld added a commit to robertoostenveld/bids-examples that referenced this pull request Apr 18, 2018
…000117_eeg_only and copied them over in an (otherwise clean) branch.
robertoostenveld added a commit to robertoostenveld/bids-examples that referenced this pull request Apr 18, 2018
@robertoostenveld
Copy link
Copy Markdown
Collaborator

this could be closed if #75 gets merged

robertoostenveld added a commit that referenced this pull request Apr 25, 2018
FIX #66 - made a clean version of the EEG-only branch
@robertoostenveld
Copy link
Copy Markdown
Collaborator

robertoostenveld commented Apr 25, 2018

The eeg-only version has been merged

sappelhoff pushed a commit that referenced this pull request May 12, 2019
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.

6 participants