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

added chapter (provisionally 23) on time patterns and author note #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

thoughtfulbloke
Copy link

Added a chapter 23 Rmd for a time based analysis (and author note as you suggested), but I didn't add the knitted files as I was more comfortable with you reviewing it first (as the primary author)

@hrbrmstr hrbrmstr self-requested a review May 12, 2018 12:46
@hrbrmstr hrbrmstr self-assigned this May 12, 2018

```{r 23_graph, message=FALSE, warning=FALSE, cache=TRUE, fig.width=8, fig.height=6}
ggplot(cyclic, aes(x=week_day, y= single_day)) +
geom_jitter(width = 0.1, alpha=0.3, size=0.3) + theme_minimal() +
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would you be amenable to adding a library(ggbeeswarm) and adding another graph plot section showing it? (i.e. same ggplot() chain but using geom_quasirandom(width = 0.1, alpha=0.3, size=0.3)?

Copy link
Owner

@hrbrmstr hrbrmstr May 12, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perhaps even a width = 0.3 for that quasirandom suggestion

image

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That visualisation looks great. Not one I had come across before, but that is part of the awesomeness of the sharing of the community & the way new perspectives show what is possible.

@hrbrmstr
Copy link
Owner

This looks 👍 !

I added one comment/suggestion but this will be helpful to folks (especially ensuring timezone b/c I tend to be a bit lazy and let things float in UTC land :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants