Skip to content

Make marimo notebooks MoLab-compatible with pinned polyplot install - #4

Merged
cursor[bot] merged 11 commits into
mainfrom
cursor/molab-notebook-imports-4d6c
Sep 11, 2026
Merged

cursor[bot] merged 11 commits into
mainfrom
cursor/molab-notebook-imports-4d6c

Conversation

@ckmah

@ckmah ckmah commented Sep 11, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • enforce Hugging Face-only dataset loading in both marimo notebooks (no raw GitHub or local fallbacks)
  • move and rename development notebook from notebook.py to dev/full_demo.py
  • simplify notebook data configuration to one variable: POLYPLOT_PARQUET_URL
  • remove private-token auth and repo/ref/file environment variable breakdown
  • add robust remote parquet loading in notebooks for http(s) URLs by downloading bytes to a temporary local parquet file before calling geopandas.read_parquet
  • reorganize packaging dependencies (pyproject.toml):
    • trim core runtime dependencies
    • add test and docs optional extras
    • keep dev group for local contributor workflows
  • add CI and manual publish workflows:
    • .github/workflows/ci.yml
    • .github/workflows/publish-test.yml
    • .github/workflows/publish.yml
    • official PyPI publish remains manual only

Validation

  • python3 -m py_compile quickstart.py dev/full_demo.py
  • ran quickstart.py in marimo edit mode with marimo-pair installed
  • executed notebook cells successfully; no notebook runtime exceptions after HTTP parquet-loader change
  • marimo snapshot confirms polyplot viewer UI rendered, including camera/grid/wireframe/opacity controls and tile source endpoint (on-demand tiles from http://127.0.0.1:8765)
Open in Web Open in Cursor 

cursoragent and others added 2 commits September 11, 2026 15:55
Co-authored-by: clarence.k.mah <clarence.k.mah@gmail.com>
Co-authored-by: clarence.k.mah <clarence.k.mah@gmail.com>
@cursor cursor Bot changed the title Make marimo notebooks work in MoLab with local polyplot imports Make marimo notebooks MoLab-compatible with pinned polyplot install Sep 11, 2026
cursoragent and others added 9 commits September 11, 2026 16:17
Co-authored-by: clarence.k.mah <clarence.k.mah@gmail.com>
Co-authored-by: clarence.k.mah <clarence.k.mah@gmail.com>
Co-authored-by: clarence.k.mah <clarence.k.mah@gmail.com>
Co-authored-by: clarence.k.mah <clarence.k.mah@gmail.com>
Co-authored-by: clarence.k.mah <clarence.k.mah@gmail.com>
Co-authored-by: clarence.k.mah <clarence.k.mah@gmail.com>
Co-authored-by: clarence.k.mah <clarence.k.mah@gmail.com>
Co-authored-by: clarence.k.mah <clarence.k.mah@gmail.com>
Co-authored-by: clarence.k.mah <clarence.k.mah@gmail.com>
@ckmah
ckmah marked this pull request as ready for review September 11, 2026 17:41
@cursor
cursor Bot merged commit 40a8ae6 into main Sep 11, 2026
1 check passed
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.

2 participants