Skip to content
This repository has been archived by the owner on May 28, 2024. It is now read-only.

plot model performance #181

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

lekoenig
Copy link
Collaborator

@lekoenig lekoenig commented Dec 16, 2022

This PR adds a file containing a few plotting functions (3_visualize/src/plotting_helpers.R). I have not tried to incorporate these into the targets pipeline yet, and so for now to run these and generate the plots you can see 3_visualize/src/plotting_script_delete_later where I lay out some pseudo-code that can ultimately be translated into targets.

Two plots are included for now:
baseline_rmse_plot

pgdl_schuyllkill_2

Within this PR, I've also committed an RDS file with the state boundaries (207 KB) so that we can omit the USAboundaries package, which is archived on CRAN and that may be way you had trouble installing within the container, @jsadler2.

@lekoenig lekoenig requested a review from jsadler2 December 16, 2022 21:21
@lekoenig lekoenig marked this pull request as draft December 16, 2022 22:18
Copy link
Collaborator

@jsadler2 jsadler2 left a comment

Choose a reason for hiding this comment

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

These changes look good to me, @lekoenig.

@lekoenig lekoenig force-pushed the 180-plot-model-performance branch from c880b29 to eae99a1 Compare March 2, 2023 21:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants