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)

@edwardchalstrey1 edwardchalstrey1 self-requested a review March 31, 2026 15:01
Copy link
Copy Markdown
Member

@edwardchalstrey1 edwardchalstrey1 left a comment

Choose a reason for hiding this comment

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

@ravjot07 Thanks very much, this works nicely 🚀

@ravjot07
Copy link
Copy Markdown
Author

@ravjot07 Thanks very much, this works nicely 🚀

Hey @edwardchalstrey1 I have submitted this as the part of my gsoc proposal, can you pl take a look at that and give me some feedback

@edwardchalstrey1
Copy link
Copy Markdown
Member

@ravjot07 Thanks very much, this works nicely 🚀

Hey @edwardchalstrey1 I have submitted this as the part of my gsoc proposal, can you pl take a look at that and give me some feedback

@ravjot07 Thanks, I just noticed you have sent us a draft proposal today - unfortunately there won't be time for us to give this draft a proper review before the formal GSoC deadline, which is also today: https://developers.google.com/open-source/gsoc/timeline - so you won't benefit from any suggested revisions we might have offered. However please feel free to submit the application via the GSoC website anyway as we'll see it there. Thanks again for your effort and we'll aim to respond properly soon.

@ravjot07
Copy link
Copy Markdown
Author

@ravjot07 Thanks very much, this works nicely 🚀

Hey @edwardchalstrey1 I have submitted this as the part of my gsoc proposal, can you pl take a look at that and give me some feedback

@ravjot07 Thanks, I just noticed you have sent us a draft proposal today - unfortunately there won't be time for us to give this draft a proper review before the formal GSoC deadline, which is also today: https://developers.google.com/open-source/gsoc/timeline - so you won't benefit from any suggested revisions we might have offered. However please feel free to submit the application via the GSoC website anyway as we'll see it there. Thanks again for your effort and we'll aim to respond properly soon.

Thanks for the response @edwardchalstrey1 will really appreciate if you can suggest some other issues that I can work on

@edwardchalstrey1
Copy link
Copy Markdown
Member

Thanks for the response @edwardchalstrey1 will really appreciate if you can suggest some other issues that I can work on

No need for this at the moment, thanks!

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

2 participants