-
Notifications
You must be signed in to change notification settings - Fork 4
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
Data Patch Pipeline and Patch 001 #69
Conversation
data/patches/patch_001
that adds the missing Southcentral Power Project data from 2013-2020.
What is the relationship between |
|
OK. I don't think that needs to stand in the way of this PR. I will make an issue to clarify this in the future. (The justification for separating scripts and data is that scripts can be tracked in GitHub, but data really shouldn't. If they are separate, then |
@ianalexmac this PR is OK with me (with the addition of the new issue) but I'm leaving approval to you because you are more familiar with the workflow. |
@ianalexmac @eldobbins actually, the inline functions used to generate statistics are kept in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work Jesse! Thank you.
This pull request incorporates a data patching methodology and a restructuring of the data folder. It also removes unused files and reorganizes things.
The web book builds great on my local machine and all figures and numbers work as expected.
In this PR is a data patch as well
data/patches/patch_001
that adds the missing Southcentral Power Project data from 2013-2020.