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

Catch up schema with changes for adding trial support. See https://gi… #173

Merged
merged 1 commit into from Jul 2, 2018

Conversation

oruebel
Copy link
Contributor

@oruebel oruebel commented Jul 2, 2018

Motivation

Catch schema up with changes for adding trial structure. See NeurodataWithoutBorders/pynwb#536

Major changes:

  • Add DynamicTable neurodata_type for column-based data tables that can be more easily expanded (by adding colmns) without having to create custom extensions
  • Add TableColmn neurodata_type to define the column of a DynamicTable data
  • Change dtype from generic number to more specific type:
    * to float forSpatialSeries.data
    * to int8 for ImageSeries.data
  • Change dtype from unspecified None to more specific type:
    * float for AnnotationSeries.data.conversion, AnnotationSeries.data.resolution, IntervalSeries.data.conversion, IntervalSeries.data.resolution attributes
    * test for AbstractFeatureSerie.data.unit,AnnotationSeries.data.unit, IntervalSeries.data.unit attributes
  • Add metadata group of type DynamicTable to Epochs type
  • Change Sex to Gender in Subject metadata
  • Add trials group for storing trial data

Note

Credit for implementing these changes goes to @ajtritt . I'm just helping with catching up the schema repo.

@oruebel
Copy link
Contributor Author

oruebel commented Jul 2, 2018

Fix #152

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.

None yet

1 participant