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

[Bug]: Compound Typechecks should Enforce Order #526

Closed
2 tasks done
lawrence-mbf opened this issue Jul 27, 2023 · 0 comments · Fixed by #530
Closed
2 tasks done

[Bug]: Compound Typechecks should Enforce Order #526

lawrence-mbf opened this issue Jul 27, 2023 · 0 comments · Fixed by #530
Assignees
@lawrence-mbf

Comments

@lawrence-mbf
Copy link
Collaborator

lawrence-mbf commented Jul 27, 2023 •
edited
Loading

What happened?

h5py (and thus pynwb and nwbinspector) returns compound types as tuples which are indexed in a particular order. In MatNWB the only checks are for the existence of the fields and not their order. For complete compatibility, the nwb field order should also be

Steps to Reproduce

Run nwbinspector on any MatNWB data that uses a compound data type. When creating the compound type, the order of the fields should not match what is listed on the nwb-schema.

Error Message

No response

Operating System

Windows

Matlab Version

r2019a

Code of Conduct
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lawrence-mbf lawrence-mbf

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Check Component Order NeurodataWithoutBorders/matnwb
1 participant
@lawrence-mbf