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

chore(NG.py): Adjust re-fetch frequency for NG.py #7714

Merged
merged 1 commit into from
Jan 11, 2025

Conversation

VIKTORVAV99
Copy link
Member

Issue

I tried re-fetching the data for NG but it created one event for every 45 min instead of one per day. So instead of 2920 re-fetch events it would have created 87600 of them if I didn't kill it.

Description

Adjust the ratio as the parsers returns a full 24 hours of data.

Double check

  • I have tested my parser changes locally with poetry run test_parser "zone_key"
  • I have run pnpx prettier@2 --write . and poetry run format in the top level directory to format my changes.

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

parsers/NG.py:78

  • Ensure there is a test case that verifies the 24-hour re-fetch frequency.
@config.refetch_frequency(timedelta(hours=24))
@github-actions github-actions bot added parser python Pull requests that update Python code labels Jan 11, 2025
@VIKTORVAV99 VIKTORVAV99 merged commit 5c68e5a into master Jan 11, 2025
23 checks passed
@VIKTORVAV99 VIKTORVAV99 deleted the vik/adjust_refetch_frequency_for_NG branch January 11, 2025 23:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
parser python Pull requests that update Python code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants