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

Do not auto-index <bids-root>/derivatives/ #184

Closed
effigies opened this issue May 24, 2018 · 1 comment
Closed

Do not auto-index <bids-root>/derivatives/ #184

effigies opened this issue May 24, 2018 · 1 comment

Comments

@effigies
Copy link
Collaborator

If I understand the situation correctly, <bids-root>/derivatives is currently indexed as if it were a standard BIDS directory. While potentially convenient, this presents some order-of-operations ambiguities for users or downstream programs that don't want to automatically include everything.

Under the recently proposed syntax, auto-indexing derivatives could be easily added by a user/developer like so:

derivs_dir = op.join(bids_dir, 'derivatives')
layout = BIDSLayout([bids_dir,
                     (derivs_dir, ['bids', 'derivatives'])])
@adelavega
Copy link
Collaborator

adelavega commented May 24, 2018

This would require adding exclude: ['derivatives/'] to bids.json. I think it's reasonable, because the "derivatives" folder is supposed to be special, and currently it's just treated as any other sub-folder of a bids_dir.

I don't think users would expect derivatives to be indexed by default, and in any case if they were, the derivatives rule set should be applied to those files.

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

2 participants