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

Reduce default network flush time #245

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

wiedehopf
Copy link

The dump1090 network outputs don't offer timing guarantees or real-time timestamps, but it is still useful for aggregators to locally timestamp messages they receive from dump1090 on a best-effort basis. With lower message rates, 500 ms of flush interval can result in significant jitter for those timestamps.
Most installs send data only on the local network or non-metered internet connections.
If necessary this default can be adjusted via --net-ro-interval by users who require a longer flush interval for some reason.

Reduce the default flush time to 90 ms to reliably flush every 2nd demodulation buffer when talking to an SDR (50 ms loop) and every loop iteration when running network-only (100 ms loop).

fixes: #244

The dump1090 network outputs don't offer timing guarantees or real-time
timestamps, but it is still useful for aggregators to locally timestamp
messages they receive from dump1090 on a best-effort basis.
With lower message rates, 500 ms of flush interval can result in
significant jitter for those timestamps.
Most installs send data only on the local network or non-metered
internet connections.
If necessary this default can be adjusted via --net-ro-interval by users
who require a longer flush interval for some reason.

Reduce the default flush time to 90 ms to reliably flush every 2nd
demodulation buffer when talking to an SDR (50 ms loop) and every loop
iteration when running network-only (100 ms loop).

fixes: flightaware#244
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.

1 participant