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

TikZ splash now uses png; demos and documentation added #136

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

pedritomelenas
Copy link
Contributor

TikZ now uses png (via pdftoppm). In this way it outputs correctly both on linux and macos. Also, github notebook previewer shows the graphics properly.

Documentation for JupyterSplashDot and JupyterSplashTikZ added.

Demo notebooks for dot (with numericalsgps) and tikz (with intpic) added.

Copy link

codecov bot commented Aug 31, 2024

Codecov Report

Attention: Patch coverage is 13.95349% with 37 lines in your changes missing coverage. Please review.

Project coverage is 26.00%. Comparing base (34b60ec) to head (773752e).

Files with missing lines Patch % Lines
gap/JupyterUtil.gi 5.55% 34 Missing ⚠️
gap/JupyterRenderable.gi 25.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #136      +/-   ##
==========================================
- Coverage   26.14%   26.00%   -0.14%     
==========================================
  Files          14       14              
  Lines         876      892      +16     
==========================================
+ Hits          229      232       +3     
- Misses        647      660      +13     
Files with missing lines Coverage Δ
gap/JupyterRenderable.gd 100.00% <100.00%> (ø)
gap/JupyterUtil.gd 100.00% <100.00%> (ø)
gap/JupyterRenderable.gi 65.51% <25.00%> (-5.32%) ⬇️
gap/JupyterUtil.gi 6.73% <5.55%> (-2.65%) ⬇️

@pedritomelenas
Copy link
Contributor Author

Added also some warnings if any of the intermediate steps to create the png (pdflatex+pdftoppm+pdfinfo+base64) fails.

@@ -1,10 +1,5 @@
InstallMethod( JupyterRender, [ IsRecord ],
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was moved to JupyterRenderable.gi
It makes more sense to be there

@@ -1,5 +1,9 @@
#! @Chapter Jupyter Utility Functions
#! @Section Functions
Copy link
Contributor Author

Choose a reason for hiding this comment

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

more descritptive title for the section

# This is still an ugly hack, but its already much better than before!
BindGlobal("JupyterSplashDot",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Used InstallGlobalFunction + declare to add documentation

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.

1 participant