Skip to content

Enhance age dataset documentation - #669

Merged
rly merged 3 commits into
devfrom
age_range
Mar 4, 2026
Merged

Enhance age dataset documentation#669
rly merged 3 commits into
devfrom
age_range

Conversation

@rly

@rly rly commented Jan 21, 2026

Copy link
Copy Markdown
Contributor

Expanded documentation for the 'age' dataset, including details on ISO 8601 Duration format and age range representation.

Copied from NWB best practices nwbinspector.readthedocs.io/en/dev/best_practices/nwbfile_metadata.html#subject

In concert with NeurodataWithoutBorders/pynwb#2161

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.

rly added 2 commits January 21, 2026 13:51
Expanded documentation for the 'age' dataset, including details on ISO 8601 Duration format and age range representation. Copied from NWB best practices nwbinspector.readthedocs.io/en/dev/best_practices/nwbfile_metadata.html#subject
@h-mayorquin

Copy link
Copy Markdown
Contributor

Thanks for this.

h-mayorquin
h-mayorquin previously approved these changes Mar 3, 2026

@h-mayorquin h-mayorquin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Comment thread core/nwb.file.yaml Outdated
Comment thread core/nwb.file.yaml Outdated
@rly
rly marked this pull request as ready for review March 4, 2026 16:23
@rly
rly requested a review from h-mayorquin March 4, 2026 16:23

@h-mayorquin h-mayorquin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@rly
rly merged commit 5ecaec1 into dev Mar 4, 2026
3 checks passed
@rly
rly deleted the age_range branch March 4, 2026 19:53
rly added a commit that referenced this pull request Jun 17, 2026
Move the OnePhotonSeries/TwoPhotonSeries documentation entry (#649) from
the 2.9.0 section to 2.10.0, since it was merged after the 2.9.0 tag.
Add the missing (#669) reference to the Subject age entry, and note that
HDMF Common 1.9.0 also promoted HERD to a stable hdmf-common data type.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.

2 participants