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

To include all rmd chunks in the reports or not? #153

Open
CarmenTamayo opened this issue May 2, 2024 · 0 comments
Open

To include all rmd chunks in the reports or not? #153

CarmenTamayo opened this issue May 2, 2024 · 0 comments

Comments

@CarmenTamayo
Copy link
Contributor

This issue relates to a discussion that started with @Bisaloo in #144, where we decided that all rmd chunks would be included in the output when knitting report templates.
I had previously omitted some of these from the outputs using include = FALSE, as I thought that, while it is useful to show external readers the code that's been used to generate estimates and tables/plots, it is probably not relevant/useful for them to see other parts of the code. For instance, the code that's been used to set the colours for the theme of the report,

theme_set(tracetheme::theme_trace())

or the code used to include graphics on the knitted report
knitr::include_graphics("transmissibility_pipeline.svg")

While these bits of code are useful for the package user, they're not necessarily also relevant for the report reader.
@Bisaloo argued that it's better to include all rmd chunks regardless, so that readers could replicate the results when copying and pasting the code chunks.

The question here is what are the priorities of users vs readers, and whether readers will have expertise or interest to run the code themselves and, if they do, whether we expect them to also have access to the .Rmd files or just to the html or pdf with the knitted report.

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

No branches or pull requests

1 participant