Skip to content

include changes of PR #84 that relate to ds000117_eeg_only#91

Closed
robertoostenveld wants to merge 3 commits intobids-standard:bep006_eegfrom
robertoostenveld:pull84-part4
Closed

include changes of PR #84 that relate to ds000117_eeg_only#91
robertoostenveld wants to merge 3 commits intobids-standard:bep006_eegfrom
robertoostenveld:pull84-part4

Conversation

@robertoostenveld
Copy link
Copy Markdown
Collaborator

No description provided.

@robertoostenveld robertoostenveld mentioned this pull request May 26, 2018
@robertoostenveld
Copy link
Copy Markdown
Collaborator Author

mac011> /Volumes/128GB/bids-validator/bin/bids-validator ds000117_eeg_only
	1: All rows must have the same number of columns as there are headers. (code: 22 - TSV_EQUAL_ROWS)
		./task-facerecognition_channels.tsv
			@ line: 2
			Evidence: name	type	units

	2: Not a valid JSON file. (code: 27 - JSON_INVALID)
		./dataset_description.json
			@ line: 11 character: 124
			Evidence: 	"SourceDatasetsURLs": {" OpenfMRI ds000117 (2014). ftp://ftp.mrc-cbu.cam.ac.uk/personal/rik.henson/wakemandg_hensonrn/", "ftp://sccn.ucsd.edu/pub/ramon/wakeman_henson_data/"}
		./task-facerecognition_eeg.json
			@ line: 29 character: 9
			Evidence: >>>>>>> 50c58bb094a79fd63be23e592de1d48ef2de36a9
		./task-facerecognition_eeg.json
			@ line: 15 character: 5
		./task-facerecognition_eeg.json
			@ line: 15 character: 5
			Evidence: <<<<<<< HEAD
		./task-facerecognition_eeg.json
			@ line: 15 character: 5
			Evidence: <<<<<<< HEAD
		./task-facerecognition_eeg.json
			@ line: 15 character: 3
			Evidence: <<<<<<< HEAD
		./task-facerecognition_eeg.json
			@ line: 15 character: 1
			Evidence: <<<<<<< HEAD

	3: This file is too small to contain the minimal NIfTI header. (code: 36 - NIFTI_TOO_SMALL)
		./sub-01/anat/sub-01_T1w.nii.gz
		./sub-02/anat/sub-02_T1w.nii.gz
		./sub-03/anat/sub-03_T1w.nii.gz
		./sub-04/anat/sub-04_T1w.nii.gz
		./sub-05/anat/sub-05_T1w.nii.gz
		./sub-06/anat/sub-06_T1w.nii.gz
		./sub-07/anat/sub-07_T1w.nii.gz
		./sub-08/anat/sub-08_T1w.nii.gz
		./sub-09/anat/sub-09_T1w.nii.gz
		./sub-10/anat/sub-10_T1w.nii.gz
		... and 6 more files having this issue (Use --verbose to see them all).

	4: First column of the channels file must be named 'name' (code: 71 - CHANNELS_COLUMN_NAME)
		./task-facerecognition_channels.tsv
			@ line: 1
			Evidence: <<<<<<< HEAD

	5: Second column of the channels file must be named 'type' (code: 72 - CHANNELS_COLUMN_TYPE)
		./task-facerecognition_channels.tsv
			@ line: 1 character: -1
			Evidence: <<<<<<< HEAD

	6: Third column of the channels file must be named 'units' (code: 73 - CHANNELS_COLUMN_UNITS)
		./task-facerecognition_channels.tsv
			@ line: 1 character: -1
			Evidence: <<<<<<< HEAD

        Summary:                 Available Tasks:        Available Modalities: 
        808 Files, 9.87MB                                T1w                   
        16 - Subjects                                                          
        1 - Session                                                            

@robertoostenveld
Copy link
Copy Markdown
Collaborator Author

this one passes just fine in the bep006_eeg branch, so there is a lot of regression in this PR. @CPernet, please advice what to do with this PR. Perhaps it is better to delete this PR; you could open an issue instead in which you describe what is to be changed in ds000117_eeg_only.

@sappelhoff
Copy link
Copy Markdown
Member

sappelhoff commented Aug 10, 2018

I think this can be closed @robertoostenveld

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