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

Uses ndx-mies extensions incorrectly #24

Open
t-b opened this issue Apr 2, 2021 · 0 comments
Open

Uses ndx-mies extensions incorrectly #24

t-b opened this issue Apr 2, 2021 · 0 comments
Labels

Comments

@t-b
Copy link
Collaborator

t-b commented Apr 2, 2021

The current ndx-MIES extension with version 0.1.0 defines the main type MIESMetaData which inherits from LabMetaData. And this requires that all additional entries are placed in /general/$name where $name has MIESMetaData neurodata type.

The fallout from this bug is that the current version of ndx-mies can not be used for reading the special MIES metadata using pynwb. Our MIES datasets and groups are not even checked with pynwb's validation.

See also t-b/ndx-MIES#6.

Possibe solution:

  • Move our MIES special stuff from /general/* to /general/MIES
  • raise ndx-MIES extension version
  • test ndx-MIES with NWBv2 files from MIES
  • tell our consumers that the location has changed
  • adapt MIES for the new location a well
@t-b t-b added the bug label Apr 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant