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

[Feature]: Improve embedding of namespaces on nwbExport #607

Open
2 tasks done
ehennestad opened this issue Oct 31, 2024 · 0 comments · May be fixed by #615
Open
2 tasks done

[Feature]: Improve embedding of namespaces on nwbExport #607

ehennestad opened this issue Oct 31, 2024 · 0 comments · May be fixed by #615
Labels
category: enhancement improvements of code or code behavior priority: medium non-critical problem and/or affecting only a small set of NWB users status: need verification potentially solved, but needs verification topic: nwb-file relates to how an exported nwb file is structured

Comments

@ehennestad
Copy link
Collaborator

ehennestad commented Oct 31, 2024 •
edited
Loading

What would you like to see added to MatNWB?

All extension namespaces in the cache (namespaces folder), will get embedded in the specs of an NWB file when running nwbExport. This can lead to files containing embedded specifications that are not relevant to any of the data types in the file.

Is your feature request related to a problem?

One recorded problem of this in #603

What solution would you like?

Implement a system detecting what namespaces a user actually used and embedding schemas off of that.

Do you have any interest in helping implement the feature?

Yes.

Code of Conduct
@ehennestad ehennestad added category: enhancement improvements of code or code behavior status: todo something needs to be done topic: matnwb-api related to improving the matnwb api priority: medium non-critical problem and/or affecting only a small set of NWB users topic: nwb-file relates to how an exported nwb file is structured and removed topic: matnwb-api related to improving the matnwb api labels Oct 31, 2024
ehennestad added a commit that referenced this issue Nov 4, 2024
@ehennestad
Only include namespaces for types that are included in NWB file on ex…
a458d62 
…port (Issue #607)
@ehennestad ehennestad linked a pull request Nov 4, 2024 that will close this issue
Draft
6 tasks
@ehennestad ehennestad added status: need verification potentially solved, but needs verification and removed status: todo something needs to be done labels Nov 4, 2024
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: medium non-critical problem and/or affecting only a small set of NWB users status: need verification potentially solved, but needs verification topic: nwb-file relates to how an exported nwb file is structured
Projects
None yet
Milestone
No milestone
1 participant
@ehennestad