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.

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

Jump to bottom

Create Generic NWBContainer class #39

Closed
ajtritt opened this issue Jun 28, 2017 · 1 comment
Closed

Create Generic NWBContainer class #39

ajtritt opened this issue Jun 28, 2017 · 1 comment
Labels
category: enhancement improvements of code or code behavior priority: high impacts proper operation or use of feature important to most users

Comments

@ajtritt
Copy link
Member

ajtritt commented Jun 28, 2017

Originally reported by: Andrew Tritt (Bitbucket: ajtritt, GitHub: ajtritt)

We need a generic NWBContainer class that users can use to read and write extensions without defining a new NWBContainer subclass.

We might need a generic TimeSeries class and a generic Interface class.
@ajtritt ajtritt added priority: high impacts proper operation or use of feature important to most users category: enhancement improvements of code or code behavior labels Aug 3, 2017
@oruebel
Copy link
Contributor

oruebel commented Aug 4, 2017

This issue has been resolved by the addition of support to dynamically create classes for extensions.

@oruebel oruebel closed this as completed Aug 4, 2017
ajtritt added a commit that referenced this issue Aug 9, 2017
@ajtritt
Merged in ecephys_fix (pull request #39)
83a72bd 
fixed bug in SpikeEventSeries constructor and added test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
category: enhancement improvements of code or code behavior priority: high impacts proper operation or use of feature important to most users
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ajtritt @oruebel