Skip to content

Conversation

@h-mayorquin
Copy link
Contributor

@h-mayorquin h-mayorquin commented Nov 21, 2025

Summary of changes

Should fix #657

Checklist

For all schema changes:

  • Add release notes for the PR to docs/format/source/format_release_notes.rst.
  • Have you included the relevant issue number using "Fix #XXX" notation where XXX is the issue number? By including "Fix #XXX" you allow GitHub to close issue #XXX when the PR is merged.
  • Make sure that hdmf-common-schema points to the latest release and not the latest commit on the main branch.

If this is the first schema change after a schema release (i.e., the version string in core/nwb.namespace.yaml does not
end in "-alpha"), then:

  • Update the version string in core/nwb.namespace.yaml and core/nwb.file.yaml to the next major/minor/patch
    version with the suffix "-alpha". For example, if the current version is 2.4.0 and this is a minor change, then the
    new version string should be "2.5.0-alpha".
  • Update the value of the version variable in docs/format/source/conf.py to the next version without the
    suffix "-alpha", e.g., "2.5.0".
  • Update the value of the release variable in docs/format/source/conf.py to the next version with the suffix
    "-alpha", e.g., "2.5.0-alpha".
  • Add a new section in the release notes docs/format/source/format_release_notes.rst for the new version
    with the date "Upcoming" in parentheses.

@h-mayorquin h-mayorquin changed the title Add units to electrode table Add micrometer units clarification to electrode table's localization coordinates Nov 21, 2025
rly added 2 commits December 10, 2025 15:49
Removed redundant specification of units for ElectrodesTable coordinate fields.
@rly
Copy link
Contributor

rly commented Dec 10, 2025

Thanks @h-mayorquin !

@rly rly enabled auto-merge (squash) December 10, 2025 23:50
@rly rly merged commit 1cbd9a8 into NeurodataWithoutBorders:dev Dec 10, 2025
2 of 3 checks passed
@h-mayorquin h-mayorquin deleted the add_units_to_electrodes_table_docs branch December 12, 2025 14:37
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.

Units for coordinates in the electrodes table

2 participants