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

Proposed fixes #34

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Proposed fixes #34

wants to merge 2 commits into from

Conversation

slarge
Copy link

@slarge slarge commented Jul 19, 2017

The pdf version has a few issues:

  • source code runs off pdf see
  • images are not resized on pdf see
  • there isn't an object called "data" should be "landings_data" see

Proposed changes modify the knitr::opts_chunk settings to fix the first 2 issues and rename the object to fix the 3rd.

Scott Large added 2 commits July 19, 2017 15:03
* source code running off pdf page
* image sizing
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

Successfully merging this pull request may close these issues.

1 participant