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

Technical review of R Markdown PDF report #130

Open
riccardoporreca opened this issue May 30, 2019 · 5 comments
Open

Technical review of R Markdown PDF report #130

riccardoporreca opened this issue May 30, 2019 · 5 comments

Comments

@riccardoporreca
Copy link
Member

riccardoporreca commented May 30, 2019

The R Markdown PDF report should be cleaned-up, so that it is technically cleaner.

Examples:

  • Review / cleanup LaTeX code in header.tex and the main report
  • library(...) statements at the beginning
  • HTML-releated statements like <br>
  • general code organization review

We probably also want to switch away from leaving metadata like title and author in the YAML header and using a customized first page, which means the PDF output does not have such properties. An option for this is to switch to a custom Pandoc template, which would be a nice maintenance refactoring, especially if we want to customize the appearance of the report in the future.

@nfarabullini nfarabullini self-assigned this Jun 3, 2019
nfarabullini pushed a commit that referenced this issue Jun 19, 2019
@gabrielfoix gabrielfoix changed the title Techincal review of R Markdown PDF report Technical review of R Markdown PDF report Jul 9, 2019
@gabrielfoix gabrielfoix added this to the Next release milestone Jul 9, 2019
@riccardoporreca
Copy link
Member Author

Details to be discussed, see premature PR #138 (review) comment.

@RolandASc
Copy link
Member

Maybe update (replace) the link to .../apps/smarp at the same time

@gabrielfoix
Copy link
Contributor

Ready to start.
Roland's comment will be addressed in another issue together with similar updates.
Talk to @riccardoporreca to define scope and approach.

@gabrielfoix gabrielfoix removed this from the Next release milestone Jul 30, 2019
@riccardoporreca
Copy link
Member Author

Maybe update (replace) the link to .../apps/smarp at the same time

Covered by #143

@riccardoporreca
Copy link
Member Author

Note that, in general, we shall reconsider if we want to keep using xelatex as engine, as well as whether PDF version (vector graphics) of the figures can be created and used instead of PNG.

Originally posted by @riccardoporreca in #152 (comment)

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

Successfully merging a pull request may close this issue.

4 participants