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

Updates to fix latex build of docs. #539

Merged
merged 1 commit into from
Jan 18, 2022
Merged

Conversation

ntouran
Copy link
Member

@ntouran ntouran commented Jan 17, 2022

The LaTeX builder did not work with SVGs so I converted them to pngs. I
also fixed some incorrect LaTeX math and dealt with a few stray
backslashes that were failing the build.

With these changes, the Sphinx LaTeX builder works properly and xelatex
builds a full pdf appropriately.

It's not a perfect PDF in that the introductory sections are added as
top level chapters to the end of the document, but other than that it is
pretty good.

Description


Checklist

  • Tests have been added/updated to verify that the new or changed code works.
  • Docstrings were included and updated as necessary
  • The code is understandable and maintainable to people beyond the author
  • There is no commented out code in this PR.

If user exposed functionality was added/changed:

  • Documentation added/updated in the doc folder.
  • New or updated dependencies have been added to setup.py.

@ntouran ntouran added the documentation Improvements or additions to documentation label Jan 17, 2022
Copy link
Member

@john-science john-science left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All the changes seem clear and straight-forward. I'm not sure I understand why we have the changes in the README.rst, but the rest is solid.

And I've seen the PDF this produces, which was good.

The LaTeX builder did not work with SVGs so I converted them to pngs. I
also fixed some incorrect LaTeX math and dealt with a few stray
backslashes that were failing the build.

With these changes, the Sphinx LaTeX builder works properly and xelatex
builds a full pdf appropriately.

It's not a perfect PDF in that the introductory sections are added as
top level chapters to the end of the document, but other than that it is
pretty good.
@ntouran ntouran merged commit db26b9c into terrapower:master Jan 18, 2022
@ntouran ntouran deleted the docfix branch January 18, 2022 16:30
ntouran added a commit to ntouran/armi that referenced this pull request Jan 20, 2022
In terrapower#539 we made some fixes to try to get LaTeX and HTML documentation
builds to both work. In doing so, the fix we made broke the badge
rendering on the main landing page of the repo on Github. Thus, this is
a different fix for that issue.

This uses a sphinx plugin to automatically convert the SVGs to pngs
before rendering and removes the .. only:: directive from the README.
ntouran added a commit that referenced this pull request Jan 20, 2022
In #539 we made some fixes to try to get LaTeX and HTML documentation
builds to both work. In doing so, the fix we made broke the badge
rendering on the main landing page of the repo on Github. Thus, this is
a different fix for that issue.

This uses a sphinx plugin to automatically convert the SVGs to pngs
before rendering and removes the .. only:: directive from the README.
scottyak pushed a commit to scottyak/armi that referenced this pull request Oct 27, 2022
The LaTeX builder did not work with SVGs so I converted them to pngs. I
also fixed some incorrect LaTeX math and dealt with a few stray
backslashes that were failing the build.

With these changes, the Sphinx LaTeX builder works properly and xelatex
builds a full pdf appropriately.

It's not a perfect PDF in that the introductory sections are added as
top level chapters to the end of the document, but other than that it is
pretty good.
scottyak pushed a commit to scottyak/armi that referenced this pull request Oct 27, 2022
In terrapower#539 we made some fixes to try to get LaTeX and HTML documentation
builds to both work. In doing so, the fix we made broke the badge
rendering on the main landing page of the repo on Github. Thus, this is
a different fix for that issue.

This uses a sphinx plugin to automatically convert the SVGs to pngs
before rendering and removes the .. only:: directive from the README.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants