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

Include docs PDF in release assets #666

Open
3 tasks
visr opened this issue Oct 13, 2023 · 6 comments
Open
3 tasks

Include docs PDF in release assets #666

visr opened this issue Oct 13, 2023 · 6 comments
Labels
CI/CD Continuous Integration/Continuous deployment documentation Improvements or additions to documentation

Comments

@visr
Copy link
Member

visr commented Oct 13, 2023

Currently we only use the Quarto HTML output which we host online. For offline use it would be good to build and include the PDF output of our documentation.

https://quarto.org/docs/output-formats/pdf-basics.html

Having this would alleviate #665, giving users a way to read old documentation.

@visr visr added documentation Improvements or additions to documentation CI/CD Continuous Integration/Continuous deployment labels Oct 13, 2023
@github-project-automation github-project-automation bot moved this to To do in Ribasim Oct 13, 2023
@Hofer-Julian
Copy link
Contributor

Sounds like a good idea. We should also check at least from time to time if the pdf is indeed complete and formatted nicely.

Having this would alleviate #665, giving users a way to read old documentation.

Agreed, a PDF is a nice alternative to hacking in a version selector or adding our own HTML page that links to the different versions (see e.g. GTK4 Rust docs)

evetion pushed a commit that referenced this issue Oct 16, 2023
Fixes #499

The process is not yet optimal, but this describes the current
situation.

I opened #666, #667 and #669 with some ideas for improvements.
@visr
Copy link
Member Author

visr commented Oct 30, 2023

I tried adding a quarto-render-pdf task in pixi.toml, which is the same as quarto-render but with --to pdf.

First it complained that no TeX installation was detected, so I ran quarto install tinytex. After that it the amsmath package failed on some equations on allocations.qmd: "Package amsmath Error: Erroneous nesting of equation structures;"

@visr
Copy link
Member Author

visr commented Nov 6, 2023

With #735 and #737 we are closer to PDF output. However, there is a bug that is fixed upstream in Quarto for including remote images, that was discussed in #737.

It's easiest to put this on hold until Quarto 1.4 is released, that is already available as a pre-release, but not in conda-forge / pixi.

@Hofer-Julian Hofer-Julian moved this from To do to Paused in Ribasim Dec 7, 2023
@visr
Copy link
Member Author

visr commented Feb 16, 2024

With the update to Quarto 1.4 in #1139 this is no longer Paused.

@visr visr moved this from Paused to To do in Ribasim Feb 16, 2024
@visr visr changed the title include docs PDF in release assets Include docs PDF in release assets Feb 16, 2024
@Hofer-Julian
Copy link
Contributor

Hofer-Julian commented Mar 20, 2024

This is currently blocked by conda-forge/quarto-feedstock#44

@SnippenE SnippenE moved this from To do to What's next in Ribasim Apr 30, 2024
@visr
Copy link
Member Author

visr commented Apr 30, 2024

This is currently blocked by #1298 (comment)

@Jingru923 Jingru923 moved this from What's next to To do in Ribasim Jul 16, 2024
@visr visr removed the enhancement label Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD Continuous Integration/Continuous deployment documentation Improvements or additions to documentation
Projects
Status: To do
Development

Successfully merging a pull request may close this issue.

2 participants