-
Notifications
You must be signed in to change notification settings - Fork 28
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
namespace checks #917
Comments
In NeurodataWithoutBorders/pynwb#1432 I've extracted the code to determine the extension namespace from the file to a separate function
|
I'm getting related errors also, would be great to get this fix integrated! |
To overcome problems like presented in dandi/helpdesk#43 this introduces solution proposed by @orugbel in #917 (comment) Unfortunately there were no release of pynwb with that function yet, so we are doomed to duplicate code and do it "manually" here for now Closes #917
Since NeurodataWithoutBorders/pynwb#1432 is not yet merged&released yet, I guess we are doomed to introduce this into dandi-cli. Submitted #1036 for our consideration ;) |
this section of the pynwb CLI figures out all the cached namespaces in an NWB file and validates against them: https://github.com/NeurodataWithoutBorders/pynwb/blob/dev/src/pynwb/validate.py#L70
the
validate
function only validates against the CORE namespace. this has resulted in this error: dandi/helpdesk#43there needs to be some refactoring in pynwb, but that's not going to fix dandi issues immediately unless we depend on the latest and greatest version. so perhaps some of the code could be imported into the dandi validate nwb function.
cc: @rly @oruebel
The text was updated successfully, but these errors were encountered: