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

refactor plot scripts #24

Open
elray1 opened this issue Jan 7, 2025 · 1 comment
Open

refactor plot scripts #24

elray1 opened this issue Jan 7, 2025 · 1 comment

Comments

@elray1
Copy link
Contributor

elray1 commented Jan 7, 2025

each model currently has its own plot script, but much of the logic is duplicated in those scripts. it would be good to clean this up. in a first pass, we could probably put a single plot script into a util folder which accepts some arguments like which plots to create and where to find forecast files to plot. In a second step, we might move some logic over to https://github.com/reichlab/idforecastutils

@matthewcornell
Copy link
Member

Good idea. This would be where we'd wrap try() around readr::read_csv() calls to capture 404 errors per #22 .

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

No branches or pull requests

2 participants