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

Gps and Pps support #1517

Open
wants to merge 180 commits into
base: main
Choose a base branch
from
Open

Conversation

krioyo
Copy link

@krioyo krioyo commented Jun 26, 2024

This is our implementation for Gps and PPS support for NTPD-rs. It also acts as a handover for the Software Project course we followed at Tu Delft.

This pr let's the user configure the port name and measurement noises for gps and pps signals. It then integrates these signals into the existing codebase to aid in determining the true time. It seeks to to close: #882

We look forward to getting a review!

kazemaksOG and others added 30 commits June 26, 2024 17:28
…s fixed and also each signal is recieved around a second(very accurate) now so that bug is also fixed
… simultaneously(each second) in which we plan to combine them before applying it to the kalman filter. This also confirms that our local clock namely pps is working as expected since the output we have matches with gps data.
…, "parse_nmea_date" and "process_gnrmc" functions.
@JamesB192
Copy link

I have yet to join the project; this is not the review you requested.

This fork is IMO underdocumented. You have 180 commits (roughly 5x the main branch's progress minus many dependency bumps), and the entire documentation is a few paragraphs in README.md. You could also label the TOML config excerpt as such; there is no indication of which GPS protocols are supported (of ~20, my guess is only a hypothetical version NMEA0183)

@rnijveld
Copy link
Member

rnijveld commented Aug 1, 2024

@JamesB192 Please be aware that this PR was a student project exploring the GPS and PPS functionality. We've worked closely with them, so we have some context that is unavailable on this PR. This code isn't ready for ntpd-rs, but I believe we were planning to look at this in the coming months. Unfortunately we only have limited budget and time to work on it.

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.

Add Pulse Per Second (PPS) Support
7 participants