Skip to content

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

Open
pedritomelenas wants to merge 7 commits into
gap-packages:masterfrom
pedritomelenas:master
Open

TikZ splash now uses png; demos and documentation added#136
pedritomelenas wants to merge 7 commits into
gap-packages:masterfrom
pedritomelenas:master

Conversation

@pedritomelenas

Copy link
Copy Markdown
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.

@codecov

codecov Bot commented Aug 31, 2024

Copy link
Copy Markdown

Codecov Report

❌ 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).
⚠️ Report is 18 commits behind head on master.

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%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pedritomelenas

Copy link
Copy Markdown
Contributor Author

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

Comment thread gap/JupyterUtil.gi
@@ -1,10 +1,5 @@
InstallMethod( JupyterRender, [ IsRecord ],

Copy link
Copy Markdown
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

Comment thread gap/JupyterUtil.gd
@@ -1,5 +1,9 @@
#! @Chapter Jupyter Utility Functions
#! @Section Functions

Copy link
Copy Markdown
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

Comment thread gap/JupyterUtil.gi
, metadata := rec() ) ) );

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

Copy link
Copy Markdown
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