bids-standard / bids-2-devel

Discussions and suggestions of backwards incompatible changes to BIDS
https://bids.neuroimaging.io/
Creative Commons Attribution 4.0 International
10 stars 1 forks source link

Make key/value pair: _run-XX suffix mandatory #33

Open tsalo opened 3 years ago

tsalo commented 3 years ago

Currently the key/value pair: _run-01, _run-02 suffix can be optionally omitted.

It would be good we me make this mandatory it would be better for long-term data collection and ease adding to data sets at a later point. For instance, you might start a study thinking you will only have 1 run and choose to omit the run suffix. Later on, your study design could change you will have multiple runs. In this situation you would have to go back and rename all of you files to include the suffix. Moreover, if this dataset has already been imported into a database (i.e LORIS), it will be ever harder to manage this.

Original authors: Unknown

tsalo commented 3 years ago

Robert Welsh wrote:

+1

yarikoptic commented 3 months ago

I do not see a reason for _run to become a special mandatory entity! We do not even have session mandatory. The overall rule I think (besides mandatory ATM sub) is to add entity whenever it is the best disambiguation from another file/data collection. As soon as there are multiple files of the same thing acquired multiple times -- add _run. If it is e.g. two different functional runs of different tasks, add _task- (IIRC it might be mandatory ATM for func but you get the point ;)).