Skip to content

Conversation

@Gold872
Copy link
Member

@Gold872 Gold872 commented Jul 27, 2025

Rework the internals of the networking library to use a Client <-> Server architecture, allowing multiple dashboards to be connected to the rover at once.

This follows what is broadcast architecture in networking, where one main server sends the same data to several clients, which in our case, is the dashboard.

TODO:

  • Unit test the multi-client functionality
  • Add separate socket for subsystems commands (which will only allow one client at a time)

@github-actions github-actions bot added networking Changes to the Networking library protobuf Changes to the auto-generated Protobuf files labels Jul 27, 2025
@github-actions github-actions bot added protobuf Changes to the auto-generated Protobuf files and removed protobuf Changes to the auto-generated Protobuf files labels Jul 31, 2025
@github-actions github-actions bot removed the protobuf Changes to the auto-generated Protobuf files label Aug 31, 2025
@github-actions github-actions bot added the protobuf Changes to the auto-generated Protobuf files label Aug 31, 2025
- Uses an entirely separate socket for sending/receiving instead of using the same one as the rover server socket
- Allows dashboard to use its own socket for timesync
- RoverSocket has its own logic now for rover-only things
- Expose RoverPacket which contains timestamp and source
- More utility methods
@github-actions github-actions bot added autonomy Changes to the Autonomy program subsystems Changes to the Subsystems program labels Nov 14, 2025
@github-actions github-actions bot added the video Changes to the Video program label Nov 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autonomy Changes to the Autonomy program networking Changes to the Networking library protobuf Changes to the auto-generated Protobuf files subsystems Changes to the Subsystems program video Changes to the Video program

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants