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

[Feature]: Report location for custom columns on units and electrode table #344

Open
2 tasks done
CodyCBakerPhD opened this issue Feb 11, 2023 · 1 comment
Open
2 tasks done
Assignees
Labels
category: enhancement improvements of code or code behavior priority: low alternative solution already working and/or relevant to only specific user(s)

Comments

@CodyCBakerPhD
Copy link
Collaborator

What would you like to see added to the NWBInspector?

Currently, we don't report the auto-parsed location for the 'special' tables of electrodes or units, because they are contained by the 'root' level, but we could add them to known locations or have special rule for returning the information

In this case I was able to recognize the custom column names, but it's possible that I might have the same column name on both tables and so would not be able to determine which table it was referring to

image

Do you have any interest in helping implement the feature?

Yes.

Code of Conduct

@CodyCBakerPhD CodyCBakerPhD added category: enhancement improvements of code or code behavior priority: low alternative solution already working and/or relevant to only specific user(s) labels Feb 11, 2023
@CodyCBakerPhD CodyCBakerPhD self-assigned this Feb 11, 2023
@CodyCBakerPhD
Copy link
Collaborator Author

Though it seems a part of this might depend on what the 'target' of the check is
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: enhancement improvements of code or code behavior priority: low alternative solution already working and/or relevant to only specific user(s)
Projects
None yet
Development

No branches or pull requests

1 participant