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

High-resolution timestamp for each sample #19

Open
7 tasks
daniel-j-h opened this issue Nov 12, 2016 · 3 comments
Open
7 tasks

High-resolution timestamp for each sample #19

daniel-j-h opened this issue Nov 12, 2016 · 3 comments

Comments

@daniel-j-h
Copy link
Collaborator

At the moment we accumulate a full 360 degree scan and return this scan to the user.

Many use-cases need high-resolution synchronization which is currently not possible on a per-sample basis to do for the user (could be estimated using motor speed and timestamp on a scan, though).

For usability reasons it would be great to have a timestamp per angle, distance sample.

Tasks:

  • time.h needs a timestamp interface
  • time_unix.c implementation
  • sweep.h needs a timestamp interface for samples (similar to get_distance, get_angle, ..)
  • sweep.c and dummy.c implementations
  • add to sweep.hpp
  • add to sweeppy
  • add to sweepjs
@kent-williams
Copy link
Contributor

Just FYI, We plan to add a timestamp to each scan packet in the future, but it first requires increasing the serial baud rate to make room for the extra bytes. I think this is a much needed addition though for the time being.

@MikeGitb
Copy link
Contributor

MikeGitb commented Feb 1, 2017

Will those time stamps be available in the next batch of devices you ship?

@MikeGitb
Copy link
Contributor

MikeGitb commented May 8, 2017

Is attaching a time stamp to each measurement package still of interest? If so, I could provide a PR.

@daniel-j-h daniel-j-h mentioned this issue Jun 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants