Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update 08-plot-ggplot2.Rmd #595

Open
wants to merge 4 commits into
base: master
from

Conversation

@sushmitavgopalan16
Copy link

sushmitavgopalan16 commented Nov 27, 2019

added more details to the explanations for answers to challenges 4a, 4b and 5

this is in response to issue # 586

added more details to the explanations for answers to challenges 4a, 4b and 5
@@ -259,6 +259,9 @@ variables and their visual representation.
> > ## Solution to challenge 4a
> >
> > Here a possible solution:
> > Notice that the `color` argument is supplied outside of the aes() function.

This comment has been minimized.

Copy link
@jcoliver

jcoliver Dec 4, 2019

Contributor

Add code formatting (backtics) to aes()

@@ -278,7 +281,11 @@ variables and their visual representation.
> > ## Solution to challenge 4b
> >
> > Here a possible solution:
> >
> > Notice that supplying the color argument inside the aes() functions enables you to

This comment has been minimized.

Copy link
@jcoliver

jcoliver Dec 4, 2019

Contributor

Add code formatting to aes() and color (and remove single quotes from the latter).

@@ -389,7 +396,9 @@ code to modify!
> > ## Solution to Challenge 5
> >
> > Here a possible solution:
> >
> > xlab() and ylab() set labels for the x and y axes, respectively

This comment has been minimized.

Copy link
@jcoliver

jcoliver Dec 4, 2019

Contributor

Add code formatting to xlab(), ylab(), and theme().

@jcoliver

This comment has been minimized.

Copy link
Contributor

jcoliver commented Dec 4, 2019

Great additions @sushmitavgopalan16 . Would you update the PR with revisions proposed above (add back-tic code formatting)?

@sushmitavgopalan16

This comment has been minimized.

Copy link
Author

sushmitavgopalan16 commented Dec 14, 2019

Thanks @jcoliver ! I've made the changes you requested!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.