Skip to content
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

Update schema to 2.2.4 #1213

Merged
merged 11 commits into from
May 28, 2020
Merged

Update schema to 2.2.4 #1213

merged 11 commits into from
May 28, 2020

Conversation

rly
Copy link
Contributor

@rly rly commented Mar 27, 2020

The current release of PyNWB, version 1.3.0, supports nwb-schema version 2.2.2. This PR adds support for nwb-schema bugfix version 2.2.3 and 2.2.4.

This PR tracks the dev branch of nwb-schema to ensure that changes in nwb-schema are compatible with PyNWB or are accompanied with corresponding changes in PyNWB. Those changes in PyNWB should be merged into this branch.

nwb-schema changelog:

  • Move nested type definitions to root of YAML files. This does not functionally change the schema.
  • Make ImagingPlane.imaging_rate optional to handle cases where an imaging plane is associated with multiple time series with different rates.
  • Add release process documentation.
  • Fix typo in nwb.ophys.yaml that prevents proper parsing of the schema.

TODO:

  • After new version of nwb-schema is released, update nwb-schema submodule to point to latest release: cd src/pynwb/nwb-schema/, git pull origin dev, git fetch --tags, git checkout [new_release], cd ../../.., git add, commit, and push

When the new version of nwb-schema is released, this branch should be merged into dev without squashing commits.

oruebel
oruebel previously approved these changes Mar 27, 2020
rly and others added 2 commits April 10, 2020 12:51
* change imaging_rate to be optional

* use kwargs for all the ImagingPlane tests

* more kwargs

* update ophys tutorial

* flake8

Co-authored-by: Ryan Ly <[email protected]>
@rly rly changed the title Update schema to 2.2.3 Update schema to 2.2.4 Apr 14, 2020
@rly rly marked this pull request as ready for review April 14, 2020 19:12
@rly rly requested review from bendichter, ajtritt and oruebel April 14, 2020 19:12
bendichter
bendichter previously approved these changes Apr 27, 2020
Copy link
Contributor

@bendichter bendichter left a comment

Choose a reason for hiding this comment

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

looks good to me. As soon as that new schema is released I think this is ready to merge into dev

@rly rly requested a review from bendichter May 28, 2020 07:43
@rly
Copy link
Contributor Author

rly commented May 28, 2020

@oruebel @ajtritt @bendichter This is good to go and should be the last PR before the release of PyNWB 1.3.1.

@rly rly merged commit c124687 into dev May 28, 2020
@rly rly deleted the schema_2.2.3 branch May 28, 2020 17:20
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