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

Timestamps as Double and printing Milliseconds #75

Merged
merged 2 commits into from
May 1, 2024

Conversation

AarjavJain101
Copy link
Contributor

  • Changed interpretation of the timestamp field from a 64 bit integer to a 64 bit precision double representing time in seconds since epoch time. (Appropriate firmware changes were made and a full radio test was done with IMU data). See note on IMU parse fails below
  • Using the datetime library I printed the timestamps with millisecond precision now.
  • Changed randomizer to create 64 bit precision double timestamps.

Example of a PARSE FAIL and millisecond precision timestamp
image

Note:

  • Some PARSE_FAILS occurred when running the system. I created a log of all the messages that were sent and noticed that sometimes the IMU message would be sent in pieces (the longer messages are the expected and 'correct' IMU messages):
41cc366dfd9a9fbe4041593c0000000d0a
41cc366dfd9a9fbe4047583fd30dae0d0a
850d0a
41cc366dfe270a
41cc366dfe270a
41cc366dfe270a
41cc366dfe270a
41cc366dfeb374bc4041583be800000d0a
41cc366dfeb374bc40415a3f7cb0000d0a

The example I chose is on the more severe end of things. I would estimate a 20% failure rate which is quite high and thus we are losing data.

Copy link
Contributor

@ishanjoshi23 ishanjoshi23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@ishanjoshi23 ishanjoshi23 merged commit bb3823b into main May 1, 2024
1 check failed
@AarjavJain101 AarjavJain101 deleted the user/Aarjavjain101/millisecond_support branch May 19, 2024 06:29
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

Successfully merging this pull request may close these issues.

None yet

2 participants