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

Some posts require packages to run #117

Open
javierluraschi opened this issue Oct 5, 2020 · 1 comment
Open

Some posts require packages to run #117

javierluraschi opened this issue Oct 5, 2020 · 1 comment

Comments

@javierluraschi
Copy link
Contributor

We should try to disable code that actually runs r code, see for instance:

```{r eval=T,echo=F}
library(dplyr)
res = data.table::fread('files/res.csv') %>%
filter(rowname %in% 'val_auc') %>% arrange(desc(V2)) %>%
rename(epoch_1 = V1, epoch_2 = V2, metric = rowname) %>%
mutate(epoch_1 = round(epoch_1,3),epoch_2 = round(epoch_2,3))
DT::datatable(res, options = list(dom = 't'))
```

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

@skeydan
Copy link
Contributor

skeydan commented Oct 6, 2020

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.

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

No branches or pull requests

2 participants