Skip to content

Conversation

@laspsandoval
Copy link
Contributor

@laspsandoval laspsandoval commented Jan 22, 2026

Change Summary

Overview

The Kiel team has requested confirmation of packet ingest within 5 minutes. The POC updated the cadence of their logs to 5 minutes to accomodate this request. This statement appears within the log every minute:

ID Description LastDataRcvd ConnectionTime Rate (kbps)
2 tlmrelay 001-00:00:00 020-19:57:14 0.0
10 Kiel 021-09:57:58 021-08:40:39 2.0

We are using the LastDataRcvd timestamp for each station (right now only Kiel) to be able to know when each packet has been received as a function of station.

Updated Files

  • calculate_ingest.py
    • parses logs

Testing

  • test_calculate_ingest.py
  • flight_iois_1.log.2026-021T10-58-00.171087

@laspsandoval laspsandoval added this to the February 2026 milestone Jan 22, 2026
@laspsandoval laspsandoval self-assigned this Jan 22, 2026
@laspsandoval laspsandoval marked this pull request as draft January 22, 2026 23:58
@laspsandoval laspsandoval changed the title I-ALiRT - Update coverage (Draft) I-ALiRT - Update coverage Jan 26, 2026
@laspsandoval laspsandoval marked this pull request as ready for review January 26, 2026 23:41
@laspsandoval laspsandoval changed the title I-ALiRT - Update coverage I-ALiRT - Update realtime packet ingest plots Jan 27, 2026
Copy link
Collaborator

@greglucas greglucas left a comment

Choose a reason for hiding this comment

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

Approving to keep it moving, but I am not sure about the current test you have with the timestamps, so that should be investigated.

Comment on lines +26 to +27
"2025-01-21T09:57:58Z",
"2025-01-21T10:27:59Z",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Something seems off here. Why is this 2025, shouldn't it be 2026 with this latest log you are using? I see 2026 timestamps in it.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants