rstudio / ai-blog

Repository for the RStudio AI Blog (formerly: TensorFlow for R Blog)
https://blogs.rstudio.com/ai/
48 stars 36 forks source link

Some posts require packages to run #117

Open javierluraschi opened 4 years ago

javierluraschi commented 4 years ago

We should try to disable code that actually runs r code, see for instance: https://github.com/rstudio/ai-blog/blob/30a4bea399d1620b02e84d98ea12fe6dda484219/_posts/2020-07-30-state-of-the-art-nlp-models-from-r/state-of-the-art-nlp-models-from-r.Rmd#L255-L262

Otherwise, when building the blog automatically, this might break or change inadvertently.

skeydan commented 4 years ago

Does this also hold for displaying images with knitr? These are the only instances where I actually evaluate R chunks, otherwise in my posts all output is static.

Should I go through the externally-contributed posts and replace any dynamic output with static results?

I'll also update the "contributing" page to say that PRs should have Rmds only, and all output should be static.