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

Shrink packet spec #24

Merged
merged 2 commits into from
Nov 6, 2024
Merged

Shrink packet spec #24

merged 2 commits into from
Nov 6, 2024

Conversation

linguini1
Copy link
Collaborator

This PR introduces changes to the packet specification which drastically reduce packet sizes and control information per packet.

The changes to the packet header include:

  • Removal of length and version fields
  • Addition of 8-bit block count field which contains the number of blocks in the packet
  • Shrink packet number to 8-bit rolling sequence number counter
  • Addition of 16 bit unsigned timestamp in minutes since power-on

Other changes:

  • Block header is just a single 8-bit type identifier
  • Debug message type removed since it was never practically used
  • Voltage ID field shrunk to 8 bit
  • Measurement time is now a 16-bit field in milliseconds to be offset from the packet header timestamp measured in minutes. This allows millisecond precision for measurement times in less space, while also keeping the maximum mission length to 1092~ hours

…is drastically shrinks packet sizes allowing more data to be transmitted with less control information.
@linguini1 linguini1 added documentation Improvements or additions to documentation enhancement New feature or request labels Nov 5, 2024
@linguini1
Copy link
Collaborator Author

radio_packet_format.pdf

Copy link

@EliasJRH EliasJRH left a comment

Choose a reason for hiding this comment

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

Some small nits, everything else good.

radio-packet-format/sections/altitude.tex Outdated Show resolved Hide resolved
radio-packet-format/sections/acceleration.tex Show resolved Hide resolved
radio-packet-format/sections/coordinates.tex Outdated Show resolved Hide resolved
@linguini1 linguini1 requested a review from EliasJRH November 6, 2024 03:17
@linguini1 linguini1 merged commit e8edd59 into main Nov 6, 2024
@linguini1 linguini1 deleted the packet-shrink branch November 6, 2024 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants