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

Implemented Qartod Timing/Gap Test #135

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

Conversation

Sakshamgupta90
Copy link
Contributor

@Sakshamgupta90 Sakshamgupta90 commented Oct 10, 2024

Implemented Timing/Gap Test as part of the IOOS QARTOD quality control checks.

Timing/Gap Test checks for the arrival of the data, ensuring that the data is arrives within the expected time window and flags cases where the data is either delayed or missing.

Code Description

The logic for the test calculates the difference between the current time now and the timestamp of the latest data tim_stmp. If this difference exceeds the expected increment tim_inc, the data is flagged as Fail.

  • A function timing_gap_test that takes the timestamp of the most recent data tim_stmp, the expected time increment tim_inc, and checks if the data has been received within the expected interval.
  • The function will return:
    • Pass (1) if the data is within the expected window.
    • Fail (4) if the data arrives later than expected.
  • Unittest were written covering following scenarios:
    • Data within the time window (expected result: pass).
    • Data outside the time window (expected result: fail).

@ocefpaf
Copy link
Member

ocefpaf commented Oct 10, 2024

@Sakshamgupta90 can you elaborate on this PR? Edit the box above to add some context of what you are implementing and how you got to the code you are submitting?

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.

2 participants