Skip to content

Commit

Permalink
[FIX] Clarify that acq_time in scans.json refers to first data po…
Browse files Browse the repository at this point in the history
…int acquired (#506)

* Clarify that acquisition time corresponds to start of acquisition.

* Change "SHOULD correspond" to "refers".

Addressing @effigies’ review.
  • Loading branch information
tsalo authored Jul 17, 2020
1 parent d27aaf1 commit 5b9be9a
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions src/03-modality-agnostic-files.md
Original file line number Diff line number Diff line change
Expand Up @@ -337,6 +337,7 @@ imaging acquisition sequence (each *run* file) within one session.
Each neural recording file should be described by at most one row.
Relative paths to files should be used under a compulsory `filename` header.
If acquisition time is included it should be under `acq_time` header.
Acquisition time refers to when the first data point in each run was acquired.
Datetime should be expressed as described in [Units](./02-common-principles.md#units).
For anonymization purposes all dates within one subject should be shifted by a
randomly chosen (but consistent across all runs etc.) number of days.
Expand Down

0 comments on commit 5b9be9a

Please sign in to comment.