Skip to content

Keep notebook output when stripping with nbstripout - #61

Closed
markbujehein wants to merge 1 commit into
easyscience:developfrom
markbujehein:fix-nbstripout-plopp-widgets
Closed

markbujehein wants to merge 1 commit into
easyscience:developfrom
markbujehein:fix-nbstripout-plopp-widgets

Conversation

@markbujehein

Copy link
Copy Markdown

Summary

The notebook-strip pixi task now runs nbstripout with the flag --keep-output, so the outputs of executed tutorial notebooks are kept and only metadata is stripped. This fixes a problem encountered when Mkdocs serve the output of certain Plopp widgets, see:

An example of the issue can currently be seen live in production here:

Add --keep-output to the notebook-strip pixi task so executed tutorial outputs are preserved and only metadata is stripped.

Fixes easyscience/dynamics-lib#277
@damskii9992 damskii9992 added [scope] enhancement Adds/improves features (major.MINOR.patch) [priority] medium Normal/default priority labels Oct 23, 2026
@AndrewSazonov
AndrewSazonov self-requested a review October 28, 2026 08:51

@AndrewSazonov AndrewSazonov left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! I’d like to keep notebooks fully stripped in Git, since plot and widget outputs can get pretty large and clutter the history.

I've opened another PR to generate the outputs in CI and render Plopp snapshots with the widget controls. It also adds a package installer cell to the notebooks linked from the docs, so they are ready to run in Colab.

Feel free to take a look at it: #63

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[priority] medium Normal/default priority [scope] enhancement Adds/improves features (major.MINOR.patch)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants