-
Notifications
You must be signed in to change notification settings - Fork 892
feat: export notebooks as PDFs via CLI #7997
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.
|
| asyncio_run(start()) | ||
|
|
||
|
|
||
| def watch_and_export_bytes( |
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.
we might be able to kill watch_and_export and consolidate to one function (and use bytes. it looks like nothing may use ExportResult.download_filename
we can do this in a followup though if you prefer
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.
Happy to do it in a followup.
|
🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.19.7-dev18 |
Bump so that `marimo export pdf ...` from marimo-team/marimo#7997 becomes available
Bump so that `marimo export pdf ...` from marimo-team/marimo#7997 becomes available
📝 Summary
Adds
marimo export pdf [OPTIONS] NAME [ARGS]...to CLI with options for output inclusion, webpdf / latex rendering mode, file watching and sandboxed notebook execution.Closes #896, closes #3415.
🔍 Description of Changes
🔍 Description of Changes
nbformat&nbconvertPDF export pipeline.--include-outputs/--no-include-outputs,--webpdf,--watch, and--sandboxfor DXwatch_and_export_bytes(kept separate fromwatch_and_exportto avoid complicating the existing text export helper with bytes/stdout/encoding concerns).test_export_pdf_missing_dependencies) to ensure missing deps produce a helpful error.marimo-team/marimo-integration-ci📋 Checklist