alexisadamsclark / FinalProject_2

Final Project for R Course II
MIT License
0 stars 2 forks source link

peer review #1

Open karlenaochoa opened 5 years ago

karlenaochoa commented 5 years ago

Hi Alex,

I forked your repo and have included comments within the Rmd! I thought you did a really nice job with your visualizations! All your plots were clear, text size and labels were large enough and easy to understand, and titles/subtitles were descriptive. All your code was straightforward and easy to follow. I have some minor suggestions for each plot - see within the document!

I really liked your use of "\n" to move axes labels off the plot, I plan to incorporate this in my work.

Nice job!

datalorax commented 5 years ago

Nice peer review, Karlena! I agree that loading all the libraries in one place is best practice, because then an outside reviewer can immediately know everything they need to run the script.

Good feedback to keep things consistent (e.g., the label of "Rank").

See here for a good discussion on dynamite plots. I actually thought about sending this out to everyone... maybe I should have.

Really great feedback!