Skip to content

Conversation

@bendichter
Copy link
Contributor

No description provided.

@bendichter bendichter requested a review from ree-gupta December 8, 2025 16:15
@bendichter bendichter changed the base branch from master to bep032 December 10, 2025 11:11
- The electrode coordinates in 3 columns (`xyz`) (use `n/a` for values if a dimension is absent).
- The ID of the probe the electrode is located on

The electrode `name` MUST be unique within the `*_electrodes.tsv` file. When a dataset contains multiple probes, electrode names can be constructed by combining the contact identifier (as provided by the probe manufacturer or principal investigator) with the `probe_name` to ensure uniqueness. For example, if two probes each have a contact labeled "1", the electrode names could be "probe01_e1" and "probe02_e1" to distinguish them.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
The electrode `name` MUST be unique within the `*_electrodes.tsv` file. When a dataset contains multiple probes, electrode names can be constructed by combining the contact identifier (as provided by the probe manufacturer or principal investigator) with the `probe_name` to ensure uniqueness. For example, if two probes each have a contact labeled "1", the electrode names could be "probe01_e1" and "probe02_e1" to distinguish them.
The electrode `name` MUST be unique within the `*_electrodes.tsv` file. When a dataset contains multiple probes, electrode names can be constructed by combining the contact identifier (as provided by the probe manufacturer or user-defined) with the `probe_name` to ensure uniqueness. For example, if two probes each have a contact labeled "1", the electrode names could be "probe01_e1" and "probe02_e1" to distinguish them.

Copy link
Member

@ree-gupta ree-gupta Dec 17, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor: "principal investigator" isn't used elsewhere in BIDS so adapted it to what is commonly used. Otherwise, looks good to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants