Skip to content

NT4 Time Sync Enhancements#8654

Draft
gerth2 wants to merge 1 commit intowpilibsuite:mainfrom
gerth2:nt4_time_sync
Draft

NT4 Time Sync Enhancements#8654
gerth2 wants to merge 1 commit intowpilibsuite:mainfrom
gerth2:nt4_time_sync

Conversation

@gerth2
Copy link
Contributor

@gerth2 gerth2 commented Mar 5, 2026

Looking to start an exploratory PR on improving time synchronization for NT4, specifically targeted toward systemcore development.

Two options on the table that I'll take a look at:

  1. Use NTP protocol proper to facilitate synchronization
  2. Port PhotonVision's algorith involving a second thread and UDP.

Option 1 would be nice if we could avoid reinventing the wheel, but critically implies it's possible to run a small NTP server on all hardware that ntcore supports. More research to come.

@gerth2 gerth2 requested a review from a team as a code owner March 5, 2026 01:26
@gerth2 gerth2 marked this pull request as draft March 5, 2026 01:26
@github-actions github-actions bot added the component: ntcore NetworkTables library label Mar 5, 2026
@PeterJohnson
Copy link
Member

The UDP approach is probably the simplest one. Done properly it can hole punch through firewalls on the client side. NTP and PTP are tempting from a standardization perspective but that does levy a complexity cost on installation.

@Gold856
Copy link
Member

Gold856 commented Mar 5, 2026

Yeah, so that likely means a revival/reboot of #7007.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: ntcore NetworkTables library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants