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

Create shortcut-tidy.R #114

Merged
merged 1 commit into from
May 26, 2016
Merged

Create shortcut-tidy.R #114

merged 1 commit into from
May 26, 2016

Conversation

Eflores89
Copy link
Contributor

Added function to "split" tidy data.frame into series understood by highcharts.

Many times, users will want to create a chart from tidy data (in the same way ggplot2 does). This basically loops around each category and creates series to add to the hc object. See the example for a better understanding.

Probably, some checks with assertthat are warranted. You can freely change anything you would do better, but personally this functionality is a big improvement to writing each series independently.

Added function to "split" tidy data.frame into series understood by highcharts.
@jbkunst
Copy link
Owner

jbkunst commented May 26, 2016

Hola @Eflores89 !

Thanks for the idea and the code! I'll merge and then add the assertthat lines, rename the function to keep the pattern and may be rename the categories parameter by group (same as ggplot aes), and obviously add you to the ctb in the DESCRIPTION file 😉.

Muchas gracias!

@jbkunst jbkunst mentioned this pull request May 26, 2016
@jbkunst jbkunst merged commit f83c829 into jbkunst:master May 26, 2016
@Eflores89
Copy link
Contributor Author

No problemo! Thanks 👍

@jbkunst
Copy link
Owner

jbkunst commented Jul 4, 2016

@Eflores89

Le podría interesar, http://rpubs.com/jbkunst/highcharter-hchart-data-frames

Saludos,

@Eflores89
Copy link
Contributor Author

Awesome! Super nifty... 👍

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

Successfully merging this pull request may close these issues.

2 participants