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

BEP030: Add fNIRS examples to master branch #333

Merged
merged 2 commits into from
Sep 1, 2022
Merged

Conversation

sappelhoff
Copy link
Member

@sappelhoff sappelhoff commented Sep 1, 2022

original additions were in #305 and #323 by @rob-luke and @robertoostenveld

rob-luke and others added 2 commits September 1, 2022 12:44
* Add fNIRS example dataset

* Use nirs branch for validator

* Update fnirs_tapping/dataset_description.json

Co-authored-by: Stefan Appelhoff <stefan.appelhoff@mailbox.org>

* Fix spacing in name

* revert link change validator

Co-authored-by: Stefan Appelhoff <stefan.appelhoff@mailbox.org>
* added another example dataset for fNIRS, see #317

* use the nirs branch of the validator from rob-luke/bids-validator

* replaced "large" PDF file with a zero-byte version

* updated fNIRRS example dataset to pass the validator

 - fixed AnatomicalLandmarkCoordinateSy_t_em
 - fixed NIRSCoordinateUnit_e_s
 - removed txt from CHANGES
 - added stimulus MATLAB m-files to bidsignore
 - added sub-95 with n/a to practicelogbook.tsv
 - use n/a for empty cells in practicelogbook.tsv
 - added practicelogbook.tsv to bidsignore - HACK
 - removed sub-77 json files from sub-75/nirs directory
 - replaced 0-byte (empty) snirf files with 1-byte (still empty) snirf files - HACK
 - use forward slashes in scans.tsv
 - inserted "type" column in all optodes.tsv files
 - reordered the columns of channels.tsv

* renamed stim to stimuli

* added SourceType back to a JSON file where I accidentally deleted it

* revert validator branch change, add entry to README

Co-authored-by: Stefan Appelhoff <stefan.appelhoff@mailbox.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants