-
Notifications
You must be signed in to change notification settings - Fork 1
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
Change modality directories to entity format #55
Comments
I think it relates also to
|
FTR to clarify the "terminology", in the schema those folders are not "modalities" which are a higher level concept, e.g. as But the point of confusion that we also have modality entity ( So the actual "thing" this issue is concerned is really some So for the bids v2 we might want to clarify this a little messy situation indeed. |
Just a note: an interesting aspect/PoV @effigies brought up in a road-trip discussion is that this |
Related to the recent #54.
Relates to many issues I've encountered with BEP016 https://github.com/bids-standard/bids-bep016; maybe best reference is bids-standard/bids-bep016#32.
The rejected bids-standard/bids-specification#1280 is also somewhat relevant.
Embedded in the current BIDS structure is a very specific hierarchy of directory structure, with very specific use or non-use of one (sometimes exclusively) "entity" key-value pairs. We might summarise as something like this:
sub-<label>/
ses-<label>/
modality/
sub-<label>_ses-<label>_suffix.ext
What I've encountered in BEP016 (and I fully expect to arise in other derivatives) is that there are circumstances in which the derivatives specific to some modality aren't just an unstructured set of data / sidecar file pairs; there can be further hierarchical structure.
In contemplating the various potential structural solutions to that, which I've probably written too much about and discouraged engagement by doing so, one factor that shows itself as an annoyance quite frequently is the modality level. Unlike subject and session, it doesn't possess the key-value formatting. So for instance, the bids-standard/bids-specification#1280 solution would look like:
sub-<label>/
ses-<label>/
modality/
model-<label>/
sub-<label>_ses-<label>_model-<label>_suffix.ext
, which is inconsistent in terms of entity key-value formatting across directories.
For complete flexibility of sub-directory structure as per #54, which would go a long way to solving the BEP016 issues, it would make far more sense if sub-directories separating data corresponding to different brain imaging modalities were to follow the entity key-value naming standard.
The text was updated successfully, but these errors were encountered: