UBC-MDS / canadian_heritage_funding

Report
https://htmlpreview.github.io/?https://github.com/UBC-MDS/canadian_heritage_funding/blob/main/doc/canadian_heritage_funding_report.html
Other
1 stars 7 forks source link

Milestone 1 feedback #49

Open andytai7 opened 2 years ago

andytai7 commented 2 years ago
  1. Exploratory data analysis in a literate code document: QUALITY

Comments You need to compare more and utilize more methods. You need to justify why some features are more important than others, be it from literature or from EDA.

  1. Exploratory data analysis in a literate code document: VIZ

Comments If figure captions are not provided the plot should be clearly explained in the text. I would recommend using figure captions.

  1. Exploratory data analysis in a literate code document: REASONING

Comments We need more rationale in terms of why you are doing certain parts in the EDA file. Why are you using some of those EDA techniques? What's the purpose of a heat map etc.? Variables lists are useless unless you define what the variables represent. Needs a higher resolution.