{"action":"created","issue":{"url":"https://api.github.com/repos/stan-dev/rstanarm/issues/53","labels_url":"https://api.github.com/repos/stan-dev/rstanarm/issues/53/labels{/name}","comments_url":"https://api.github.com/repos/stan-dev/rstanarm/issues/53/comments","events_url":"https://api.github.com/repos/stan-dev/rstanarm/issues/53/events","html_url":"https://github.com/stan-dev/rstanarm/pull/53","id":126729411,"number":53,"title":"Make HTML vignettes self-contained so they can work on CRAN","user":{"login":"yihui","id":163582,"avatar_url":"https://avatars.githubusercontent.com/u/163582?v=3","gravatar_id":"","url":"https://api.github.com/users/yihui","html_url":"https://github.com/yihui","followers_url":"https://api.github.com/users/yihui/followers","following_url":"https://api.github.com/users/yihui/following{/other_user}","gists_url":"https://api.github.com/users/yihui/gists{/gist_id}","starred_url":"https://api.github.com/users/yihui/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yihui/subscriptions","organizations_url":"https://api.github.com/users/yihui/orgs","repos_url":"https://api.github.com/users/yihui/repos","events_url":"https://api.github.com/users/yihui/events{/privacy}","received_events_url":"https://api.github.com/users/yihui/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-01-14T19:27:59Z","updated_at":"2016-01-15T02:25:37Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/stan-dev/rstanarm/pulls/53","html_url":"https://github.com/stan-dev/rstanarm/pull/53","diff_url":"https://github.com/stan-dev/rstanarm/pull/53.diff","patch_url":"https://github.com/stan-dev/rstanarm/pull/53.patch"},"body":"I'm not sure why you preferred `self_contained = FALSE`, but for vignettes to be viewed correctly on CRAN. It will be much easier for CRAN maintainers to copy your vignettes without taking care of external image files.\r\n\r\nNote I also changed the R CMD INSTALL option `--precleanOOPS` to `--preclean`. I don't know what the former means. It does not seem to be a standard option.\r\n\r\nThe default file size of the `html_document` is large due to the Bootstrap dependencies, and you can reduce the vignette sizes significantly by using `html_vignette`. Let me know if you want to do that and I can amend this PR."},"comment":{"url":"https://api.github.com/repos/stan-dev/rstanarm/issues/comments/171849912","html_url":"https://github.com/stan-dev/rstanarm/pull/53#issuecomment-171849912","issue_url":"https://api.github.com/repos/stan-dev/rstanarm/issues/53","id":171849912,"user":{"login":"bgoodri","id":3386282,"avatar_url":"https://avatars.githubusercontent.com/u/3386282?v=3","gravatar_id":"","url":"https://api.github.com/users/bgoodri","html_url":"https://github.com/bgoodri","followers_url":"https://api.github.com/users/bgoodri/followers","following_url":"https://api.github.com/users/bgoodri/following{/other_user}","gists_url":"https://api.github.com/users/bgoodri/gists{/gist_id}","starred_url":"https://api.github.com/users/bgoodri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bgoodri/subscriptions","organizations_url":"https://api.github.com/users/bgoodri/orgs","repos_url":"https://api.github.com/users/bgoodri/repos","events_url":"https://api.github.com/users/bgoodri/events{/privacy}","received_events_url":"https://api.github.com/users/bgoodri/received_events","type":"User","site_admin":false},"created_at":"2016-01-15T02:25:37Z","updated_at":"2016-01-15T02:25:37Z","body":"Thanks @yihui ! The motivation for self_contained = FALSE was that CRAN doesn't like more than 5MB of vignettes but we wanted to put a lot of images based on MCMC output into the HTML vignettes. With svglite and external images, this got us down to 3.5 MB but everything else we tried was putting us over. I can't remember if we tried `html_vignette`, but most of the bulk was due to the images. I had hoped that the script CRAN uses to push the vignettes to the website would read vignettes/.install_extras but apparently it does not.\r\n\r\nThe `--precleanOOPS` thing was just a hack we came up with accelerate development because RStudio wanted to add the `--preclean` argument unless it thought the option was already (mis)specified. With the `--preclean` argument, it rebuilds the autogenerated content in rstanarm/src/ even when the developer does not need to at that moment. R CMD INSTALL just ignores the `--precleanOOPS` argument with a warning."}}