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

unsupported features from the r cookbook #161

Open
14 of 41 tasks
chriddyp opened this issue Jan 27, 2015 · 5 comments
Open
14 of 41 tasks

unsupported features from the r cookbook #161

chriddyp opened this issue Jan 27, 2015 · 5 comments
Labels

Comments

@chriddyp
Copy link
Member

Here are some remaining unsupported features as taken from the great ggplot2 cookbook.

The cookbook examples are in this branch: https://github.com/ropensci/plotly/tree/add-r-cookbook-tests, in this folder: https://github.com/ropensci/plotly/tree/add-r-cookbook-tests/tests/cookbook-test-suite

@tdhock
Copy link
Contributor

tdhock commented Feb 16, 2015

I checked off the items that I have fixed in #163

@tdhock
Copy link
Contributor

tdhock commented Mar 10, 2015

use this table to keep track of which cookbook test files from https://github.com/ropensci/plotly/tree/add-r-cookbook-tests/tests/cookbook-test-suite

which have been ported to testthat files under https://github.com/ropensci/plotly/tree/master/tests/testthat

original file ported by to file branch/PR
axes.R tdhock test-cookbook-axes.R #172
bars_and_lines.R cpsievert test-ggplot-bar.R #200
distributions.R cpsievert test-ggplot-[histogram density].R
facets.r
legends.R
lines.R
means_and_error_bars.R
multiple_graphs_on_one_page.r
scatterplots.R cpsievert test-cookbook-scatterplots.R #183
titles.R

@cpsievert
Copy link
Collaborator

It seems like a good number of unimplemented cookbook tests overlap with tests already implemented testhat side. For example, see here and here.

Unless anyone objects, I'm going to favor putting cookbook examples under test-ggplot-*.R (instead of test-cookbook-*.R) whenever appropriate

@chriddyp @mkcor @tdhock.

@cpsievert
Copy link
Collaborator

Taking a closer look, I don't think it's really worth adding any of the examples in bars_and_lines.R. We already have tests for most of it. There are a couple examples that have components that we don't currently test for like theme_bw() and theme(legend.position = ...), but those things have nothing to do with bars/lines.

@jackparmer
Copy link
Contributor

Pretty sure this is outdated now and superseded by https://github.com/ropensci/plotly/milestone/3

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

No branches or pull requests

4 participants