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

Change NTD ridership schedule to run once a week #3556

Merged
merged 1 commit into from
Nov 22, 2024

Conversation

erikamov
Copy link
Contributor

Description

This PR changes the schedule of the NTD monthly ridership scrape (sync_ntd_data_xlsx DAG ) to run once a week (every Monday) instead of every day.
There is no need to run every day since it is a monthly file, but having the process running once a week give us a better window to add and process new files once it is released.

Part of the issue #3519.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation

How has this been tested?

Tested running airflow locally.

Post-merge follow-ups

  • No action required
  • Actions required (specified below)

After merge, check the schedule for sync_ntd_data_xlsx to see if the next run is for Monday.

instead of every day.

There is no need to run every day since it is a
monthly file, but having the process
running once a week give us a better window to
add and process new files once it is released.

[#3519]
Copy link
Member

@charlie-costanzo charlie-costanzo left a comment

Choose a reason for hiding this comment

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

Awesome, thank you!! LGTM

@erikamov erikamov merged commit 75ff729 into main Nov 22, 2024
1 check passed
@erikamov erikamov deleted the 3519-change-NTD-ridership-schedule branch November 22, 2024 19:56
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