-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add physio data #9
Conversation
To maximize BIDS compliance we should add a top-level
Biopac Also add more verbose columns descriptions ( such as describing ECG or EDA or PPG electrode types/ placement on the body/ signal-specific amplifier module... and specifying units if we can have it) with additional fields as in this example in the spec:
|
For the columns name + description, it is not possible to comply to the BIDS standards because they actually do not account for different type of cardiac signals (i.e. PPG vs ECG). Also, they don't provide a standard column name for EDA... |
IIUC, these are the column name so for us:
which should have a top-level json with something like:
BTW: why do we have a column "time". |
My 2 cents on the issue of the time column: it is negligible in terms of data size, and easier to parse for users than some kind of meta-data start time + sampling time. If we end up dropping frames as part of preprocessing, this format can also accomodate irregular grids. Is it something that is covered by the BIDS extension? |
@all-contributors please add @me-pic for data preparation |
I've put up a pull request to add @me-pic! 🎉 |
Add segmented raw physio data (PPG, ECG, EDA, RESP, TTL channels):