-
Notifications
You must be signed in to change notification settings - Fork 892
improvement: use vega 6 mimetype, capture screenshots on pdf export #7906
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| "application/vnd.vega.v5+json", | ||
| "application/vnd.vegalite.v6+json", | ||
| "application/vnd.vega.v6+json", | ||
| ]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc @Light2Dark for export-to-pdf
manzt
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Are the vega specs kept in the repr mimebundle for the ipynb exports? I know that is separate from these changes, but some renders will support that directly on top of PNG.
|
@manzt yes, we augment the outputs with the PNG (instead of overwrite) |
|
Great. |
|
🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.19.5-dev25 |
…arimo-team#7906) This makes sure we use the correct vega/vegalite mime type (now 6) and well as updates some logic to include also include vega charts when capturing screenshots.
This makes sure we use the correct vega/vegalite mime type (now 6) and well as updates some logic to include also include vega charts when capturing screenshots.