Feedback on Milestone 1

Nazliozum commented 5 years ago

@olivia-lin @ksm45 You have made lots of great additions to your analysis for this milestone.

Here are some remaining issues:

[x] Add a link to your report on the main readme
[x] Add instructions to run your analysis on main readme
[x] Add dependencies with their version information on the readme
[x] Fix the broken links in the Dataset section on your readme
[ ] Try to change the proposal structure on the main readme. I can also suggest not to use future tense in your statements. Same thing applies for your report.
[x] Add figure numbers to your plots in the report. Try to resize them as some figures are too big. Try to elaborate on the visualizations. You should have some explanations as to what the figures are telling us.
[x] Decrease bullet points in your report. You can definitely write explanations in paragraph format for some sections such as the Limitations and Future Directions sections.
[x] You have a References section, but I don't see any references. Either delete the section or add your sources.

Suggestions:

mani-kohli commented 5 years ago

fixed readme script link

mani-kohli commented 5 years ago

added dependencies