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

Solution to Plotly Issue #3828 for the new version of Kaleido #243

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

Conversation

konmenel
Copy link

@konmenel konmenel commented Jan 21, 2025

A while ago I implemented a fix for the same issue in PR #168. However, I saw that Kaleido had some major updates and it no longer needs to be compiled with Chromium. I suspected that the fix would be even easier with the new code so I had a look.

The change is 1 line which adds a script tag in the HTML to configure the stroke in Mathjax. This is hardcoded which might not be ideal, but it is very easy to move the config in a separate file if need be (something like what I did in the old RP).

Hopefully, this will close Plotly's issue 196

Issues that I encountered

While I was testing I noticed two issues which I list below:

  1. Chrome headless was not loading similar to issue Image export fails #241. I already posted my solution in the discussion of the issue.
  2. Two pages appear in the pdf, issue Two pages appear in pdf output #235. I haven't investigated but one solution which is not ideal is to generate an SVG instead. Converting from SVG to PDF is very simple.

Note

I use Arch btw

@ayjayt
Copy link
Collaborator

ayjayt commented Jan 21, 2025

Thanks for doing this, I'll talk a look over today and tomorrow.

@ayjayt
Copy link
Collaborator

ayjayt commented Jan 29, 2025

Yeah, this looks good. We have some wide changes coming in so I may have to integrate it manually. Thanks for doing this @konmenel

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.

My LaTeX strings look bold when exporting in .pdf and .eps
2 participants