-
Notifications
You must be signed in to change notification settings - Fork 159
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
[SCHEMA] Add derivatives entities to the schema #645
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks reasonable. I've added a little bit of extra content so they don't look quite so bare.
Co-authored-by: Chris Markiewicz <effigies@gmail.com>
Is it possible to enumerate which ones are referred to as E.g. |
@adam2392 We could add a field to the schema that describes scope, but I think that could quickly become confusing (e.g., |
Oh okay yeah that sounds good! I suppose that's a pretty simple fix for now and makes sense given what you said. Is |
This stems from mne-tools/mne-bids#449 (comment).
Changes proposed:
res
,den
,label
, anddesc
) to the schema.space
entity so it works for EEG/MEG/iEEG and MRI data.