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

[Add Check]: Datasets are not empty #324

Open
2 tasks done
CodyCBakerPhD opened this issue Dec 7, 2022 · 0 comments · May be fixed by #403
Open
2 tasks done

[Add Check]: Datasets are not empty #324

CodyCBakerPhD opened this issue Dec 7, 2022 · 0 comments · May be fixed by #403
Assignees
@alessandratrapani
Labels
category: new check a new best practices check to apply to all NWBFiles and their contents

Comments

@CodyCBakerPhD
Copy link
Contributor

What would you like to see added to the NWBInspector?

From NeurodataWithoutBorders/pynwb#1002, check that datasets are not empty.

Maybe need to consider edge case of ImageSeries with external file format, memory doesn't reclal if the Dataset there is None, empty, or if the data attribute is not set at all in that case.

Pseudocode or other logic of the check

No response

Do you have any interest in helping implement the check function?

Yes.

Code of Conduct
@CodyCBakerPhD CodyCBakerPhD added the category: new check a new best practices check to apply to all NWBFiles and their contents label Dec 7, 2022
@alessandratrapani alessandratrapani self-assigned this Sep 18, 2023
@alessandratrapani alessandratrapani linked a pull request Sep 18, 2023 that will close this issue
Draft
4 tasks
@CodyCBakerPhD CodyCBakerPhD linked a pull request Sep 18, 2023 that will close this issue
[New Check]: TimeSeries with empty dataset #403
Draft
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alessandratrapani alessandratrapani

Labels
category: new check a new best practices check to apply to all NWBFiles and their contents
Projects
None yet
Milestone
No milestone
2 participants
@CodyCBakerPhD @alessandratrapani