Skip to content

ci: strip notebook outputs with nbstripout#8

Merged
Marius1311 merged 3 commits into
mainfrom
chore/add-nbstripout
Jun 22, 2026
Merged

ci: strip notebook outputs with nbstripout#8
Marius1311 merged 3 commits into
mainfrom
chore/add-nbstripout

Conversation

@Marius1311

Copy link
Copy Markdown
Member

Changes

  • Add the nbstripout pre-commit hook (rev 0.9.1) so Jupyter outputs (embedded base64 figures, execution counts) no longer enter git.
  • One-time strip of outputs from all 2 tracked notebooks: 1.9 MB → 0.0 MB in the working tree.

Why

Committed notebook outputs were the dominant source of repo bloat and made notebook diffs/merges unreviewable. Stripping inputs-only keeps history lean going forward; figures/data are already gitignored.

Notes

  • Two commits: the hook (reviewable) and the mechanical bulk strip.
  • No history rewrite — existing large blobs stay in history; this only stops new ones. Branch is cut from origin/main.
  • Local working-tree notebooks were left untouched (done in an isolated worktree); outputs there reappear only after you pull + re-run.

Closes #

Marius1311 and others added 3 commits June 19, 2026 17:14
Stops Jupyter outputs (embedded base64 figures) from entering git, which
keeps diffs reviewable and prevents the repo history from bloating.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Mechanical, generated by the new nbstripout hook over all 2 tracked
notebooks. No input cells changed.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Provides the nbstripout CLI in the pixi env for manual stripping,
alongside the pre-commit hook. Lock refreshed by pixi as usual.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@Marius1311 Marius1311 merged commit 21d30a3 into main Jun 22, 2026
1 check failed
@Marius1311 Marius1311 deleted the chore/add-nbstripout branch June 22, 2026 08:48
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