-
-
Notifications
You must be signed in to change notification settings - Fork 544
Updates to R docs #213
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
Updates to R docs #213
Conversation
cc @chriddyp |
@cpsievert - can we still keep an option for printing the object as an |
```{r} | ||
f <- plotly_POST(p, filename = "r-docs/midwest-boxplots") | ||
class(f) | ||
f | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hey @cpsievert - this doesn't seem to work for me:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, sorry, you'll need this devtools::install_github("ropensci/plotly#296")
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added that in plotly/plotly.R@350b582 (that pull request - plotly/plotly.R#296 - is ready to merge, BTW) |
Actually, thanks to this post, it might not be too bad to make htmlwidgets play nicely with Jekyll |
sweet. I merged this into #215, so I'm going to close this and merge the other one. |
Just after I made all these changes, I remembered this consequence of using htmlwidgets. So, I’m not sure how exactly md -> html happens, but most likely it's gonna require a messy hack to embed these graphs client-side. :(