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

New Reporting Template #82

Closed
OskaratOmni opened this issue Apr 19, 2024 · 16 comments · Fixed by #83
Closed

New Reporting Template #82

OskaratOmni opened this issue Apr 19, 2024 · 16 comments · Fixed by #83
Assignees

Comments

@OskaratOmni
Copy link
Collaborator

Hey @dgkeyes and @tvroylandt
(David sorry for the double posting with email too)

We have a new template that incorporates some updates to OMNI style.

I'm hoping that we can model report templates in pdf, html, and word for the OMNI R package after this general style template.

Let me know what you think.

thanks!
Report Template 2024-specs.pdf

@dgkeyes
Copy link
Member

dgkeyes commented Apr 23, 2024

@OskaratOmni just chiming in here to say that Thomas and I talked. He should be able to do the PDF template in the next couple weeks. We can then work on the other formats. We'll comment here with updates/questions.

@OskaratOmni
Copy link
Collaborator Author

Sounds good @dgkeyes , thanks!

tvroylandt added a commit that referenced this issue Apr 30, 2024
tvroylandt added a commit that referenced this issue Apr 30, 2024
tvroylandt added a commit that referenced this issue Apr 30, 2024
tvroylandt added a commit that referenced this issue May 2, 2024
tvroylandt added a commit that referenced this issue May 2, 2024
tvroylandt added a commit that referenced this issue May 2, 2024
@tvroylandt
Copy link
Collaborator

@dgkeyes @OskaratOmni I did a first draft on this one in the dedicated branch.
Please tell me what you think of it

skeleton.pdf

@tvroylandt tvroylandt linked a pull request May 2, 2024 that will close this issue
@OskaratOmni
Copy link
Collaborator Author

Hey @tvroylandt , this looks very good, thank you.

Two questions -

  • in the template, is it fairly easy to re-position that grey sidebar as needed? I assume its just a css style that can be moved around readily?
  • the figures both just say TRUE for a caption. I know this is easy to adjust but for the sake of the template could you please have an actual place-holder caption in each case? Anything will do ... like "Sepal length data" and "Yes, colored triangles" or whatever comes to mind.

Thanks!

@tvroylandt
Copy link
Collaborator

  • The sidebar is just a blockquote inside a column div so you can put it anywhere you want
  • True, done

tvroylandt added a commit that referenced this issue May 2, 2024
@OskaratOmni
Copy link
Collaborator Author

Hey @tvroylandt how do I view the updated version of this skeleton?
I see the commit but am not sure how to view the pdf. Thanks!

@tvroylandt
Copy link
Collaborator

Either you can test it on the branch 82-new-reporting-template

skeleton.pdf

Or I put it here. In general we avoid to push pdf, html and so because it's getting big for the Github repo and it throws errors for the R package.

@dgkeyes
Copy link
Member

dgkeyes commented May 8, 2024

@OskaratOmni if you need help with this, just let us know.

@OskaratOmni
Copy link
Collaborator Author

Hey @tvroylandt , one thing I just noticed on the template is that it looks like the text under the first header isn't all the way left-aligned. Do you see what I mean? The text under first header is slightly indented compared to text under second and third level headers.

Otherwise, I think we are ready to start trying this one out.

What are the next steps for making it useable by our team? Do you push to main branch and we can access it from templates like the others?

Thanks!
Oskar

tvroylandt added a commit that referenced this issue May 16, 2024
@tvroylandt
Copy link
Collaborator

It was the padding of the column. I will merge it and you can use it as it is yes

@dgkeyes
Copy link
Member

dgkeyes commented May 16, 2024

@OskaratOmni we have merged this so at this point, you can access it just like the other templates. Please have your team try it out and let us know what you see. If/when you want us to make this template in Word and/or HTML formats, please feel free to open a separate issue and we can work on that.

@OskaratOmni
Copy link
Collaborator Author

Sounds good @dgkeyes thanks!
I have played with it a little bit and the only minor issue I'm having is that I get warnings like this:

Warning in grid.Call(C_textBounds, as.graphicsAnnot(x$label), x$x, x$y,  :
  font family not found in Windows font database
Warning in grid.Call(C_textBounds, as.graphicsAnnot(x$label), x$x, x$y,  :
  font family not found in Windows font database
Warning in grid.Call.graphics(C_text, as.graphicsAnnot(x$label), x$x, x$y,  :
  font family not found in Windows font database
Warning in grid.Call.graphics(C_text, as.graphicsAnnot(x$label), x$x, x$y,  :
  font family not found in Windows font database
Warning in grid.Call.graphics(C_text, as.graphicsAnnot(x$label), x$x, x$y,  :
  font family not found in Windows font database

that show up in the pdf when I knit. They are from the first ggplot call. I couldn't identify any fonts that I don't have loaded - the only one I see is Calibri, so I'm puzzled about the warning.

I didn't have time to work on it too hard so I just added a "warning = FALSE" to the knitter chunk options, which works fine. I'm fine with that solution but if you know of something else to check that would cause the font family warning, then please let me know.

@dgkeyes
Copy link
Member

dgkeyes commented May 20, 2024

Hmm, strange. Can you try this please? https://show.rfor.us/v3tQN0Rb

@OskaratOmni
Copy link
Collaborator Author

Hey @dgkeyes ,
I was about to start a new issue because i've been having a bunch of data viz problems lately, and most of them were when I was calling theme_omni but lots of just kind of weird behavior. I did a bunch of package updates today and the problems were still there.
I couldn't think of anything else so I switched the graphics device backend from AGG back to default and most or all of the data viz problems have gone away. Figures that wouldn't even render are now rendering. Could be a mac thing?
Anyway, my guess is that that was not the expected resolution so I just wanted you to know.
best
Oskar

@dgkeyes
Copy link
Member

dgkeyes commented Jun 17, 2024

Hmm, that's strange. Might be worth the two of us getting on a call to go over this and the issues you're seeing in #85. Do you want to schedule something?

@OskaratOmni
Copy link
Collaborator Author

OskaratOmni commented Jun 19, 2024

@dgkeyes, scheduled!
I did a little googling today and this is the closest thing i found, re; issue 85:
rstudio/rmarkdown#2180

I tried reverting to an older version of htmltools, which is of course pretty easy to do. But when I tried knitting with that older version installed it immediately required me to update to the current version anyway. So that wasn't super productive, lol.

@dgkeyes dgkeyes reopened this Sep 11, 2024
@dgkeyes dgkeyes closed this as completed Jan 6, 2025
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 a pull request may close this issue.

3 participants