acep-uaf / aetr-web-book-2024

Alaska Electricity Trends Report as a web book
https://acep-uaf.github.io/aetr-web-book-2024/
Creative Commons Attribution Share Alike 4.0 International
0 stars 4 forks source link

Verify that scripts directory is now unneeded and delete it. #70

Closed eldobbins closed 2 weeks ago

eldobbins commented 2 months ago

In reference to #69 which moves the action to data/: The patches in data/ and data/main.R may render things in scripts/ like scripts/main.R obsolete. There needs to be some picking through to make sure we don't lose anything important if we do delete scripts that are no longer relevant.

Optional twists:

(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 data/ can be listed in .gitignore while scripts/ is tracked normally. Not really an issue with our tiny dataset - thinking of the future.)

jikaczmarski commented 2 months ago

One major dependency is the scripts/inline_functions folder. This contains functions written in R for calling statistics in text. We just need to maintain this somewhere as it's pretty important. It can move with fairly limited refactoring elsewhere.

ianalexmac commented 2 months ago

Please don't delete scripts/ it contains code vital to the build.

eldobbins commented 2 weeks ago

This is an enhancement that is being obsoleted by all the other ways we are generating data to support reports in the future. Will close without working on it.