Abdullah has a well laid out main repository README file with a good introduction and links to relevant homework pages.
This may just be a stylistic choice but I personally would have linked from the main README to the HW03 README file, instead of directly to the gapminder exploration.
I would also suggest placing a link from your HW03.md page back to the README page(s) for improved browsability.
Extremely thorough progress report in your HW03 README, well done there.
ggplot2/dplyr exploration:
I would recommend a more descriptive title for this page.
Get the maximum and minimum of GDP per capita for all continents
You have done a great job on the table and plot, but the source code you used to make it is hidden. There's just html in the raw text version, but no explanation of how you created this data.
Look at the spread of GDP per capita within the continents
Now the source code is visible! Nice plot here with good use of ggplot elements including: facet_wrap and theme.
Possible improvements I would suggest for the plot here:
change plot labels from scientific notation using options(scipen=5)
Change x labels, y labels, and title using labs(x = "x label here", y = "y label here", title = "title here").
I'm not sure why your table didn't properly load. I wasn't even able to run your code chunk when I tested it. However, it may have something to do with summarize_each which Rstudio complains is depreciated.
I think the following code chunk properly loads your table, though its very wide:
#### Compute a trimmed mean of life expectancy...
- Interesting weighing system and nice table using `pandoc`. Well built plot, nice section!
#### How is life expectancy changing over time on different continents?
- Really nice looking plot here! Good use of `theme` and `labs` function. However, colouring by continent when everything is already faceted by country is redundant. You could have coloured by, for example, `log10(gdpPercap)`.
- Again, unfortunately your table did not load properly. It seems that markdown doesn't like to load tables within tables!
#### Report the absolute and/or relative abundance of countries with low life expectancy...
- Interesting benchmark used here, it proved to be very useful!
- I would recommend not including such excessively long tables. Perhaps use the `head` function to show just a sample of your data with a table.
- Really cool plot here, creativly arranged!
Overall a great job. Clearly a lot of effort put into this, doing much more on the project that was required. Next time just proofread your work to ensure everything looks the way its supposed to!
Great job on the assignment Abdullah! Here is my peer review for your efforts.
Your README.md is organized neatly and it sure gives a good overview of your repository.
I really like how you explain your approach on the different tasks in your README file. That's a cool way to do it, I will definitely consider doing that myself at some point.
Your code is very clear on concise, which sure makes the life easier for your reviewers
I would have to say that I agree with @HScheiber in that "placing a link from your HW03.md page back to the README page(s) for improved browsability." but again, that just a personal opinion and your approach is also fine.
You did a really impressive job with your plots. They look amazing, also you utilized an interesting approach that I really like.
It's obvious that you feel comfortable working with tasks like these, you have done more than what was required. You demonstrate finesse and thorough knowledge.
Is there a obvious reason for you keeping .RMD versions of your README files?
Lastly, I really like what you did in your last part of the assignment Report the absolute and/or relative abundance of countries with low life expectancy... You did an awesome job with this data. Not only does it look amazing, also the data is really interesting. Good job!
Keep up the amazing work and good luck with your future tasks!
Main Repository/README:
Abdullah has a well laid out main repository README file with a good introduction and links to relevant homework pages.
This may just be a stylistic choice but I personally would have linked from the main README to the HW03 README file, instead of directly to the gapminder exploration.
I would also suggest placing a link from your HW03.md page back to the README page(s) for improved browsability.
Extremely thorough progress report in your HW03 README, well done there.
ggplot2/dplyr exploration:
Get the maximum and minimum of GDP per capita for all continents
Look at the spread of GDP per capita within the continents
Now the source code is visible! Nice plot here with good use of ggplot elements including:
facet_wrap
andtheme
.Possible improvements I would suggest for the plot here:
options(scipen=5)
labs(x = "x label here", y = "y label here", title = "title here")
.I'm not sure why your table didn't properly load. I wasn't even able to run your code chunk when I tested it. However, it may have something to do with
summarize_each
which Rstudio complains is depreciated.I think the following code chunk properly loads your table, though its very wide:
knitr::kable(spread_gdp_continent_table, align = 'c')