Skip to content

Add sketch vectorization notebook#51

Merged
akshayka merged 4 commits into
mainfrom
aka/fix-sketch-vectorization-not-wasm
Apr 9, 2026
Merged

Add sketch vectorization notebook#51
akshayka merged 4 commits into
mainfrom
aka/fix-sketch-vectorization-not-wasm

Conversation

@akshayka

@akshayka akshayka commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Pulls in the sketch-to-SVG walkthrough from rambip/sketch-vectorization@1c53779 as notebooks/external/sketch-vectorization.py.
  • Adapts the data-loading cell so the notebook works both in WASM (via yarl.URL, as before) and in non-WASM (a tiny _DataDir wrapper lazily downloads pinned data files to a temp cache so load_normalized / show_example see plain Paths).
  • Pins dependencies and declares onnxruntime only for non-emscripten via a PEP 508 marker, so Pyodide doesn't try to install it.

Test plan

  • uvx marimo check notebooks/external/sketch-vectorization.py
  • uv run scripts/create-sessions.py notebooks/external/sketch-vectorization.py runs end-to-end (ONNX inference + optimizer for all 13 gallery examples)
  • uv run scripts/validate-sessions.py — all sessions clean and up-to-date
  • Open in molab and confirm WASM still works

akshayka added 4 commits April 9, 2026 12:22
Pulls in the sketch-to-SVG walkthrough from rambip/sketch-vectorization
and adapts the data loading so it works both in WASM and locally. The
original used a filesystem path relative to the source repo; the
non-WASM branch now lazily downloads data files from the pinned commit
into a temp cache so load_normalized/show_example see plain Paths.
@akshayka akshayka enabled auto-merge (squash) April 9, 2026 19:33
@akshayka akshayka disabled auto-merge April 9, 2026 19:34
@akshayka akshayka merged commit c2bd24f into main Apr 9, 2026
2 checks passed
@akshayka akshayka deleted the aka/fix-sketch-vectorization-not-wasm branch April 9, 2026 19:34
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