Skip to content

Add micrometer units clarification to electrode table's localization coordinates #658

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.

Sign up for GitHub

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

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
+8 −6
Open

Add micrometer units clarification to electrode table's localization coordinates #658

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
7adab48
add units to electrode table
h-mayorquin Nov 21, 2025
837d70c
release notes
h-mayorquin Nov 21, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions core/nwb.ecephys.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -288,17 +288,17 @@ groups:
- name: x
neurodata_type_inc: VectorData
dtype: float32
doc: x coordinate of the channel location in the brain (+x is posterior).
doc: x coordinate of the channel location in the brain (+x is posterior). Units should be specified in microns.
quantity: '?'
- name: y
neurodata_type_inc: VectorData
dtype: float32
doc: y coordinate of the channel location in the brain (+y is inferior).
doc: y coordinate of the channel location in the brain (+y is inferior). Units should be specified in microns.
quantity: '?'
- name: z
neurodata_type_inc: VectorData
dtype: float32
doc: z coordinate of the channel location in the brain (+z is right).
doc: z coordinate of the channel location in the brain (+z is right). Units should be specified in microns.
quantity: '?'
- name: imp
neurodata_type_inc: VectorData
Expand All @@ -313,17 +313,17 @@ groups:
- name: rel_x
neurodata_type_inc: VectorData
dtype: float32
doc: x coordinate in electrode group
doc: x coordinate in electrode group. Units should be specified in microns.
quantity: '?'
- name: rel_y
neurodata_type_inc: VectorData
dtype: float32
doc: y coordinate in electrode group
doc: y coordinate in electrode group. Units should be specified in microns.
quantity: '?'
- name: rel_z
neurodata_type_inc: VectorData
dtype: float32
doc: z coordinate in electrode group
doc: z coordinate in electrode group. Units should be specified in microns.
quantity: '?'
- name: reference
neurodata_type_inc: VectorData
Expand Down
2 changes: 2 additions & 0 deletions docs/format/source/format_release_notes.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,8 @@ Minor changes
- Made ``EventDetection.times`` optional and deprecated. Use `source_idx` instead. (#620)
- Clarified documentation of ``PlaneSegmentation`` that at least one of ``image_mask``, ``pixel_mask``,
and ``voxel_mask`` is required. (#636)
- Specified that units for ``ElectrodesTable`` coordinate fields (``x``, ``y``, ``z``, ``rel_x``, ``rel_y``, ``rel_z``)
should be in microns. (#658)

2.8.0 (November 24, 2024)
-------------------------
Expand Down