Skip to content

Fix/latex deps#59

Open
ravjot07 wants to merge 2 commits intogambitproject:mainfrom
ravjot07:fix/latex-deps
Open

Fix/latex deps#59
ravjot07 wants to merge 2 commits intogambitproject:mainfrom
ravjot07:fix/latex-deps

Conversation

@ravjot07
Copy link
Copy Markdown

Closes #41.
The docs recommended a full TeX Live install (texlive-full, on the order of ~1.5 GB) for PDF/PNG/SVG. The project only needs a small slice of that stack, and the generated document was loading newpxtext / newpxmath unconditionally, which pulls in a heavy font stack on many systems.

What changed
latex_wrapper in core.py — newpx is now optional. The wrapper uses \IfFileExists{newpxtext.sty}{...}{} so Palatino-style fonts load when present and compilation still succeeds with default fonts when they are not.
CI (.github/workflows/test.yml) — Replaced texlive-full with texlive-pictures, texlive-latex-extra, plus ghostscript and pdf2svg so CI mirrors a lean setup and runs faster with less download.

How to verify
pytest tests/ (as in CI)

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.

[Good first issue] Reduce LaTeX dependency to the minimal required

1 participant