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

Feature/export plots #276

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

Feature/export plots #276

wants to merge 7 commits into from

Conversation

tbukic
Copy link

@tbukic tbukic commented Jul 19, 2024

Added possibility to export plots to .png files. Since package requires very specific combination of dependency versions to run, selenium can not be used directly with bokeh and panel commands for saving plots. Plots are exported to .html, and shot-scraper CLI tool is used to screenshot them. Screenshots are post-processed to crop bokeh toolbar and add title if necessary.
Exporting is added to all pipeline notebook cells which plot anything.

tbukic and others added 7 commits July 18, 2024 16:39
…with 'if interactive' or with 'if save_plots'. save_plots is a new boolean variable at the start of notebook. No more displaying plots by just writing variable name at the end of cell, all those are now encapsulated by display(plot), otherwise they wouldn't work inside 'if' clause.
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