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

Automate hospital admission patch #2043

Open
wants to merge 29 commits into
base: main
Choose a base branch
from

Conversation

aysim319
Copy link
Contributor

@aysim319 aysim319 commented Aug 29, 2024

Description

automate patching for hospital-admission and change backfill merge cadence from 28 days to 1 month worth of data

Associated Issue(s)

@aysim319
Copy link
Contributor Author

aysim319 commented Sep 9, 2024

Documenting comments here not mentioned in the PR.

@minhkhul mentioned in dm: we need to consider patches if the outages are more than n_checked days?

There would be a cascading effect of having to fixing the subsequent merged parquet files. Not sure if that level of outage should be handled in an automated fashion due to touching many more backfill files.

Second thing @minhkhul pointed out: need to consider outage days where the date falls between the start or the end date.

Totally valid edge case that I forgot to consider and need to check for the edge case.

@aysim319 aysim319 force-pushed the automate-hospital-admission-patch branch from 0ee004d to 74bebe3 Compare November 6, 2024 16:12
@aysim319 aysim319 removed the request for review from melange396 November 13, 2024 20:50
@nmdefries nmdefries mentioned this pull request Dec 6, 2024
@nmdefries nmdefries requested a review from minhkhul December 10, 2024 22:21
@aysim319 aysim319 linked an issue Feb 5, 2025 that may be closed by this pull request
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.

Patch 07/31/2024 data for hospital admission
3 participants