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

rvdss interface and new fn layout so current/historical data can be easily fetched #1551

Merged
merged 11 commits into from
Nov 22, 2024

Conversation

nmdefries
Copy link
Contributor

@nmdefries nmdefries commented Oct 15, 2024

Summary:

Implements interface for rvdss indicator. Change fn structure. Makes historical data pulling use up-to-date fn args. Marks additional work with "TODO: ". This won't work entirely as-is. At minimum, fn imports need to be updated for the new run.py script.

Some files have been renamed. Looks like that's causing a conflict with the historical fns file. Fixed

Prerequisites:

  • Unless it is a documentation hotfix it should be merged against the dev branch
  • Branch is up-to-date with the branch to be merged with, i.e. dev
  • Build is successful
  • Code is cleaned up and formatted

@nmdefries nmdefries requested a review from cchuong October 15, 2024 17:45
@nmdefries nmdefries changed the title Ndefries/rvdss framework rvdss interface and new fn layout so current/historical data can be easily fetched Oct 15, 2024
old_detection_data.to_csv('season_2023_2024/' + RESP_COUNTS_OUTPUT_FILE,index=True)
old_positive_data.to_csv('season_2023_2024/' + POSITIVE_TESTS_OUTPUT_FILE,index=True)
for i, base_url in enumerate(included_urls):
dict_list[i]["weekly"] = fetch_dashboard_data(url, 2023)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

issue: Need to make sure key is one of the three standard keys ("county", "positive", etc)

import os

from delphi.epidata.acquisition.rvdss.utils import get_weekly_data, get_revised_data, get_dashboard_update_date
from delphi.epidata.acquisition.rvdss.constants import DASHBOARD_BASE_URL, RESP_DETECTIONS_OUTPUT_FILE, POSITIVE_TESTS_OUTPUT_FILE, COUNTS_OUTPUT_FILE
Copy link
Contributor Author

Choose a reason for hiding this comment

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

issue: check imports

@nmdefries nmdefries merged commit 30f3df6 into add_rvdss_indicator Nov 22, 2024
5 of 6 checks passed
@nmdefries nmdefries deleted the ndefries/rvdss-framework branch November 22, 2024 18:35
@cchuong cchuong restored the ndefries/rvdss-framework branch December 3, 2024 22:48
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