search

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

Store date/time information in the JSON sidecar instead of _scans.tsv file #27

Open tsalo opened 3 years ago

tsalo commented 3 years ago

See https://github.com/rordenlab/dcm2niix/issues/93

Original authors: @chrisgorgo

tsalo commented 3 years ago

@gllmflndn wrote:

All of the information stored in _scans.tsv files could be stored instead in JSON sidecar files. Given the potential redundancy, as mentioned in https://github.com/rordenlab/dcm2niix/issues/93#issuecomment-299329243, I would consider removing _scans.tsv from the specification.

tsalo commented 3 years ago

Personally, I would prefer to see the scans tsv restructured a bit (see #29) rather than removed. Extracting an acquisition time that applies to a whole set of files from the same acquisition is difficult with jsons, and could be much easier with an altered tsv file. Additionally, the tsv file is a great place for scan-level information like QC ratings.

robertoostenveld commented 3 years ago

The scans.tsv file is also very useful for experimental details that vary from scan to scan. For example, the subjects alertness/fatigue ratings or some other experimental variable that is not assessed by the scanner but by the experimenter in another way. That is proper data in itself, not metadata about the recording.