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

Error when MRI files have space entity? #1059

Closed
adam2392 opened this issue Sep 8, 2020 · 2 comments
Closed

Error when MRI files have space entity? #1059

adam2392 opened this issue Sep 8, 2020 · 2 comments

Comments

@adam2392
Copy link
Member

adam2392 commented Sep 8, 2020

I don't think the spec prevents MRI files from having space entity, but I got this error when running the validator:

	1: [ERR] Files with such naming scheme are not part of BIDS specification. This error is most commonly caused by typos in file names that make them not BIDS compatible. Please consult the specification and make sure your files are named correctly. If this is not a file naming issue (for example when including files not yet covered by the BIDS specification) you should include a ".bidsignore" file in your dataset (see https://github.com/bids-standard/bids-validator#bidsignore for details). Please note that derived (processed) data should be placed in /derivatives folder and source data (such as DICOMS or behavioural logs in proprietary formats) should be placed in the /sourcedata folder. (code: 1 - NOT_INCLUDED)
		./sub-la02/ses-presurgery/anat/sub-la02_ses-presurgery_space-ACPC_T1w.nii
			Evidence: sub-la02_ses-presurgery_space-ACPC_T1w.nii
		./sub-la02/ses-presurgery/anat/sub-la02_ses-presurgery_space-orig_T1w.nii
			Evidence: sub-la02_ses-presurgery_space-orig_T1w.nii

Is this intended based on https://bids-specification.readthedocs.io/en/derivatives/05-derivatives/03-imaging.html#preprocessed-coregistered-andor-resampled-volumes ?

If not, where does this occur and perhaps I can make a PR?

@effigies
Copy link
Collaborator

effigies commented Sep 9, 2020

space is not currently part of BIDS raw, but is used for derivatives. If this is a derivatives dataset, then we still have yet to fully implement support.

@sappelhoff
Copy link
Member

Feel free to reopen with more info if Chris' answer did not solve your issue @adam2392

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

No branches or pull requests

3 participants