search

apang00 / Energy-Policy

0 stars 2 forks source link

Peer Review #4 #26

Open Crooksyyy opened 4 months ago

Crooksyyy commented 4 months ago

Readme

Need to update introduction to repo good file structure Good LLM explanation

Other folder

Need to update sketches as they are not relevant to this paper LLM usage.txt is not updated as the llm explanation states in readme figures should include captions to be independent items

Data Folder

appears well done

Scripts

Simulate data is not complete or relevant Download data is not complete or relevant Test data is not complete or relevant Model script is not used or completed replication script is not complete - use same name structure as other files 2 data cleaning files, delete the one that is not used

Data cleaning

No preamble - what data is being cleaned? what files do i need to replicate this what order do I need to clean these files? Use preamble or name structure to do this Remove commented install.packages Missing comments in code so it is unclear why or what is being done throughout

Paper

Great Intro missing link to code in paper pdf figures missing proper referencing in written sections figures missing captions Otherwise incomplete but a great start

Crooksyyy commented 4 months ago

Also need to update repo name to something descriptive