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

arrange date correctly #7

Merged
merged 10 commits into from
Jan 24, 2025
Merged

arrange date correctly #7

merged 10 commits into from
Jan 24, 2025

Conversation

donga0223
Copy link
Collaborator

Added arrange(date) to sort the data file in chronological order.

Copy link
Member

@nickreich nickreich left a comment

Choose a reason for hiding this comment

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

Thanks for adding the README edits from NYC. Things look pretty good, but I have a few minor requested changes to the target-data files.

README.md Outdated

The Flu MetroCast Hub makes public forecasts for New York City, and is open to adding forecasts for other jurisdictions that have public data. For New York City, the influenza-related data are Emergency Department (ED) visits due to Influenza-like Illness (ILI), but other data sources could be modeled as well, such as the percentage of outpatient visits with a primary complaint of ILI or count of all hospitalizations due to influenza.
The Flu MetroCast Hub makes public forecasts for New York City (NYC) and is open to adding forecasts for other jurisdictions with near real-time public-facing data. For New York City, the influenza-related data are daily Emergency Department (ED) visits due to influenza-like illness (ILI), which includes ED chief complaint mention of flu, fever, and sore throat. Other data sources, whewe available, could be modeled as well, such as the percentage of outpatient visits with a primary complaint of ILI or count of all hospitalizations due to influenza.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
The Flu MetroCast Hub makes public forecasts for New York City (NYC) and is open to adding forecasts for other jurisdictions with near real-time public-facing data. For New York City, the influenza-related data are daily Emergency Department (ED) visits due to influenza-like illness (ILI), which includes ED chief complaint mention of flu, fever, and sore throat. Other data sources, whewe available, could be modeled as well, such as the percentage of outpatient visits with a primary complaint of ILI or count of all hospitalizations due to influenza.
The Flu MetroCast Hub makes public forecasts for New York City (NYC) and is open to adding forecasts for other jurisdictions with near real-time public-facing data. For New York City, the influenza-related data are daily Emergency Department (ED) visits due to influenza-like illness (ILI), which includes ED chief complaint mention of flu, fever, and sore throat. Other data sources, where available, could be modeled as well, such as the percentage of outpatient visits with a primary complaint of ILI or count of all hospitalizations due to influenza.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

One more thing about this: Sharon also wanted to add citations at the end of this page, formatted like this:
New York City Department of Health and Mental Hygiene. EpiQuery - Syndromic Surveillance Data. [Clarify date or cadence for accessing]. https://nyc.gov/health/epiquery
I haven’t included it yet. Could you add this with the correct format?

src/clean_target_data.R Show resolved Hide resolved
target-data/time-series.csv Outdated Show resolved Hide resolved
target-data/oracle-output.csv Outdated Show resolved Hide resolved
Copy link
Member

@nickreich nickreich left a comment

Choose a reason for hiding this comment

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

This all looks good to me! Thanks for all this work.

@nickreich nickreich merged commit e5be471 into main Jan 24, 2025
@nickreich nickreich deleted the target_data branch January 24, 2025 21:28
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