-
Notifications
You must be signed in to change notification settings - Fork 190
[ENH] BEP045: "Peripheral" physiology (raw data) #2267
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
base: master
Are you sure you want to change the base?
Conversation
I'm not sure if a corresponding change should be made to modalities.yaml, I'm having trouble finding where the latter table is used by the compiler macros.
Incorporate physio BEP section 4 to physiological-recordings.md
Add physio as a datatype
Some checks on formatting (particularly for how I entered the allowable values) might be necessary here, I also added some comments on the BEP document to flag potential issues or changes we could make going forward.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should at least partially address the validation failures.
(I'm assuming that technical schema comments are more appropriate here than the regular discussion channels you mentioned. LMK if you'd rather they go somewhere else.)
| description: | | ||
| Amplifier settings during data acquisition (e.g. gain, sampling filters, bandwidth, etc.) | ||
| Strings MAY be represented in the format Setting:Value (e.g. Gain:10) | ||
| type: array of strings |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| type: array of strings | |
| type: array | |
| items: | |
| type: string |
| from participant to recording device. A pseudonym can also be used to | ||
| prevent the equipment from being identifiable, so long as each | ||
| pseudonym is unique within the dataset. | ||
| type: string or array of strings |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| type: string or array of strings | |
| anyOf: | |
| - type: string | |
| - type: array | |
| items: | |
| type: string |
| the sensor, connecting cables, and amplifiers. If more than one manufacturer | ||
| applies, list the manufacturer for each piece of hardware, in order | ||
| of attachment from participant to recording device. | ||
| type: string or array of strings |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| type: string or array of strings | |
| anyOf: | |
| - type: string | |
| - type: array | |
| items: | |
| type: string |
| produced the measurements upstream of the recording device, including | ||
| the sensor, connecting cables, and amplifiers. Equipment should be listed | ||
| in order of attachment, from participant to recording device. | ||
| type: string or array of strings |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| type: string or array of strings | |
| anyOf: | |
| - type: string | |
| - type: array | |
| items: | |
| type: string |
This BEP proposes an expansion of the raw physiological data section of the standard.
The main development is happening in https://github.com/physiopy/bids-specification-physio
To see the rule of engagements, see the discussion board.
To leave comments, engage with the physiopy repository (specifically, with its PR #32) rather than with this PR.
However, we're not really ready for reviews or comments yet, we are opening a draft just to have the HTML and schema json preview. We will communicate ASAP when we are ready for public engagement.