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

[New Check]: Add check electricalseries of dtype int16 #408

Draft
wants to merge 9 commits into
base: dev
Choose a base branch
from

Conversation

alessandratrapani
Copy link
Collaborator

@alessandratrapani alessandratrapani commented Sep 27, 2023

Fix #395

Add check on ElectricalSeries data type: if the conversion and offset are DEFAULT values and the data type is any integer ("uint16", "int16", etc).

  • add check function
  • add unit tests
  • add check function documentation in ecephys.rst
  • update CHANGELOG.md

@alessandratrapani alessandratrapani added the category: new check a new best practices check to apply to all NWBFiles and their contents label Sep 27, 2023
@alessandratrapani alessandratrapani self-assigned this Sep 27, 2023
CHANGELOG.md Outdated Show resolved Hide resolved
@CodyCBakerPhD CodyCBakerPhD changed the title Add check electricalseries of dtype int16 [New Check]: Add check electricalseries of dtype int16 Sep 29, 2023
@CodyCBakerPhD
Copy link
Contributor

Just occurred to me while inspecting some files on the GUIDE; we should also consider the channel_conversion array in this check

If the attribute exists, and the elements of it are not also 1.0, then it's fine to have conversion=1.0 and offset=0.0 because the array of conversion factors is the intended way of specifying that information

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: new check a new best practices check to apply to all NWBFiles and their contents
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Add Check]: ElectricalSeries of dtype int16 with conversion=1.0 and offset=0.0 is potentially wrong?
2 participants