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

Read session start time from header #75

Open
ageorgou opened this issue Nov 13, 2020 · 0 comments
Open

Read session start time from header #75

ageorgou opened this issue Nov 13, 2020 · 0 comments
Labels
data Handling of LabVIEW or other data to-discuss Needs discussion with research group

Comments

@ageorgou
Copy link
Contributor

We've had a FIXME comment in the code for this, but it's now becoming relevant in one of the tests too!

Each NWB file must have a session start time. Normally, we get this from the speed data. Where that is not available, we use the current time. Howeve, the start time may also be recorded in the header, so we could use that instead of the current time. As a side-effect, this will give a consistent signature for the test.

Even when the speed data is available, it is possible for the header to have a different experiment time recorded (as in this comment). Which one should take precedence in those cases?

@ageorgou ageorgou added data Handling of LabVIEW or other data to-discuss Needs discussion with research group labels Nov 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data Handling of LabVIEW or other data to-discuss Needs discussion with research group
Projects
None yet
Development

No branches or pull requests

1 participant