-
Notifications
You must be signed in to change notification settings - Fork 206
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
allow pandoc option setting from saveWidget #81
Comments
Same issue here for Karl Broman iplot (testjs package). Possible solution seems to be setting selfcontained=FALSE |
Thanks for reporting this. We will investigate the issue and figure out potential solutions to fix it. |
The problem can be related to the size of the data being placed in the widget. With R/qtlcharts, I get this stack space overflow error only for the library(qtl)
library(qtlcharts)
library(htmlwidgets)
data(grav)
grav$pheno <- grav$pheno
grav <- calc.genoprob(grav)
out <- scanone(grav, phe=1:nphe(grav), method="hk")
# this is okay
out_plot1 <- iplotMScanone(out, lodcolumn=1:103)
saveWidget(out_plot1, file="tmp1.html")
# this gives "stack space overflow"
out_plot3 <- iplotMScanone(out, lodcolumn=1:104)
saveWidget(out_plot3, file="tmp3.html") |
This should be fixed now. See here |
Excellent! |
agreed! #ty for this! On Thu, Jul 9, 2015 at 12:10 PM, Karl Broman notifications@github.com
|
BTW I've subsequently discovered that it's best to increase the pandoc So you shouldn't require fiddling with the arguments manually any longer. On Thu, Jul 9, 2015 at 3:39 PM, boB Rudis notifications@github.com wrote:
|
When trying to prep some examples for a blog post for a new widget I tripped the following error:
It doesn't happen when knitting, though, so this may just be a weird edge case.
to reproduce (but a restart of RStudio also cleared the error at least one time for me).
UPDATE
It's pretty reproducible, but even using the extra parameters from the command line (bash, pandoc 1.13.x, os x 10.10.x) didn't solve the issue. I think it requires compiling pandoc to support rts options. ugh.
folks might just want to be careful when making ginormous vis.
The text was updated successfully, but these errors were encountered: