-
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
Replace _scans.tsv with _recordings.tsv #29
Comments
@jgrethe wrote: Or perhaps _data.tsv which would be more domain agnostic |
For me it is also not clear what precisely should go in the In case of task-fMRI with events, a single line (with the In case of EEG with recorded/digitized electrode positions, the |
I would propose "acquisitions" though I will also advocate for #27, which would render this issue irrelevant. |
In Common Principles, acquisition refers to "a continuous uninterrupted block of time during which a brain scanning instrument was acquiring data according to particular scanning sequence/protocol." I actually think adopting "acquisitions" for the tsv file would be a great idea, @jbteves, independent of #27. Here's my reasoning:
|
There is a need also very similar to
so there With @neuromechanist we are working out |
Recently, we started using |
Good point, and 👍 on such generalization. Let's think alternative name |
Right now the
_scans.tsv
file keeps track of which acquisitions exist within a particular session. The word “scans” is a little domain-specific, especially as people begin to write BIDS formats for things like electrophysiology. Why not call thisrecordings
, as this is more domain-general and still reflects the same overall idea?Original authors: Unknown
The text was updated successfully, but these errors were encountered: