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 request: Manually defined timestamps #19

Open
caniko opened this issue Jan 25, 2022 · 1 comment
Open

Feature request: Manually defined timestamps #19

caniko opened this issue Jan 25, 2022 · 1 comment

Comments

@caniko
Copy link

caniko commented Jan 25, 2022

As of now the timestamping is done on the OpenEphys side, we need to define the timestamps of incoming frames the moment they arrive to Bonsai to remove this variable latency.

This latency is not a problem on LED setups; however, we use DeepLabCut. Thresholding images is faster than GPU-based DNN predictions (DLC live benchmarks). We want to make sure that the latency introduced by DeepLabCut is inngeligible.

I was planning to work on a PR that allows an optional argument to be sent from Bonsai storing the timestamp the moment the frame arrives. What do you think about overloading the timestamp-generating function(s)?

@alejoe91
Copy link
Contributor

Hi @caniko

I don't have time to work on this unfortunately.
Feel free to make a PR! I'd be happy to review it ;)

I think that the best solution would be to store the Bonsai timestamp as an extra field in the binary message. This would allow, for posprocessing, to have both the OE timestamps and the Bonsai timestamps and to make benchmark comparisons.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants