Skip to content

[burt_network] BURT Network 3.0#44

Merged
Gold872 merged 23 commits into
mainfrom
burt-network-3
May 25, 2026
Merged

[burt_network] BURT Network 3.0#44
Gold872 merged 23 commits into
mainfrom
burt-network-3

Conversation

@Gold872

@Gold872 Gold872 commented Jul 27, 2025

Copy link
Copy Markdown
Member

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
@Gold872 Gold872 marked this pull request as ready for review May 25, 2026 23:29
@Gold872 Gold872 requested a review from a team as a code owner May 25, 2026 23:29

@lregueiferos lregueiferos left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@Gold872 Gold872 merged commit 42da985 into main May 25, 2026
4 checks passed
@Gold872 Gold872 deleted the burt-network-3 branch May 25, 2026 23:37
Gold872 added a commit to BinghamtonRover/Dashboard that referenced this pull request May 26, 2026
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