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

Figure wrapping code for air zone reports #4

Open
wants to merge 54 commits into
base: fig-wrapping
Choose a base branch
from

Conversation

karawoo
Copy link
Collaborator

@karawoo karawoo commented Mar 29, 2019

This has everything from #3, plus code to attempt wrapping text around figures (currently used in 05_georgia_strait.Rmd only as a test case). It uses the wrapfigure package for LaTex and defines a custom knitr chunk hook that generates LaTeX code for wrapped figures. However it will need further customization based on the actual figure sizes, length of text, etc. to look good -- right now either we have large empty spaces on some pages, or figures that overlap or appear out of order with the text.

karawoo and others added 30 commits March 5, 2019 22:06
One report (Georgia Strait) with 4 plots + helper functions and script to
generate report output
Co-Authored-By: karawoo <karawoo@users.noreply.github.com>
There is an issue with how the air zone management level column gets
colored (part of the header gets colored as well), need to look into this
Only 24h; need to add columns for annual mean still
Will need to tweak figure appearance for some of them
Requires compiling with lualatex or xelatex
Captions are now functions in the functions.R script
karawoo added 24 commits March 17, 2019 23:01
Depends on changes in this branch: bcgov/pm25-caaqs-indicator#8
No real reason to have them separate
No need to make sure there is room for the "CAAQS" label
fig.env = "wrapfigure" is not sufficient, because \begin{wrapfigure} requires
some additional options
No longer need to join with stations data frame because the necessary info is
now included in the annual data
@ateucher
Copy link
Contributor

Thanks @karawoo - this is great to have!

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.

None yet

2 participants