Skip to content

fix(demo): install coordinode-embedded from PyPI wheel, drop rustup/Rust build#64

Merged
polaz merged 1 commit intomainfrom
fix/#63-pypi-install-notebooks
May 3, 2026
Merged

fix(demo): install coordinode-embedded from PyPI wheel, drop rustup/Rust build#64
polaz merged 1 commit intomainfrom
fix/#63-pypi-install-notebooks

Conversation

@polaz
Copy link
Copy Markdown
Member

@polaz polaz commented May 3, 2026

Problem

All 4 Colab notebooks installed coordinode-embedded by:

  1. Installing Rust via rustup (~2 min)
  2. Installing maturin (~30 sec)
  3. Building coordinode-embedded from git source (~5 min)

Total: ~8 minutes of compilation on every new Colab runtime.

Fix

Replace the entire rustup/Rust/maturin/protoc block with a single pip install coordinode-embedded from PyPI. The release workflow publishes pre-built manylinux wheels for Linux x86_64 (which is what Colab runs), so installation takes ~30 sec.

Also updated README timing note and markdown descriptions in all 4 notebooks.

…ust build

Replace the ~8 min Rust compilation path (rustup + maturin + git source)
with a simple pip install from PyPI. coordinode-embedded ships pre-built
manylinux wheels so Colab users get the package in ~30 sec with no Rust
toolchain required. Also update README timing note.
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 3, 2026

📝 Walkthrough

Summary by CodeRabbit

  • Documentation
    • Updated Google Colab setup instructions across README and demo notebooks. Pre-built dependency installation now completes in ~30 seconds on first run (versus prior ~5-8 minutes), with instant subsequent runs. Installation process simplified.

Walkthrough

This PR updates the README and four demo notebooks to reflect that Google Colab now installs coordinode-embedded via pre-built PyPI wheels (~30 sec) rather than building from source (~5–8 min), removing Rust toolchain, protoc, and maturin build steps and replacing them with a simple conditional pip install.

Changes

Colab Setup Simplification

Layer / File(s) Summary
Documentation Update
README.md
Colab "first cell" note now states installation uses pre-built wheels (~30 sec) instead of source build (~8 min).
Notebook Dependency Installation
demo/notebooks/00_seed_data.ipynb, demo/notebooks/01_llama_index_property_graph.ipynb, demo/notebooks/02_langchain_graph_chain.ipynb, demo/notebooks/03_langgraph_agent.ipynb
Colab setup cells replace Rust/protoc/maturin build logic with a unified pkgs list and single pip install call; conditionally prepends coordinode-embedded when in Colab without a COORDINODE_ADDR environment variable.
Notebook Environment Documentation
demo/notebooks/00_seed_data.ipynb, demo/notebooks/01_llama_index_property_graph.ipynb, demo/notebooks/02_langchain_graph_chain.ipynb, demo/notebooks/03_langgraph_agent.ipynb
Opening markdown sections are updated to describe pre-built wheel installation (~30 sec) instead of source compilation (~5–8 min); minor comment/wording adjustments in embedded-mode adapter and connection sections.
Notebook Formatting & Cleanup
demo/notebooks/00_seed_data.ipynb, demo/notebooks/02_langchain_graph_chain.ipynb, demo/notebooks/03_langgraph_agent.ipynb
Markdown headers are reformatted (e.g., step numbering); verification output display adjusted; cleanup calls client.close() instead of graph.close().

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix(demo): install coordinode-embedded from PyPI wheel, drop rustup/Rust build' accurately summarizes the main change: replacing Rust compilation with PyPI pre-built wheels for faster Colab setup.
Description check ✅ Passed The description clearly explains the problem (8 minutes of compilation), the fix (pip install from PyPI wheels), and its impact (~30 second installation), directly relating to all changeset modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/#63-pypi-install-notebooks

Review rate limit: 3/5 reviews remaining, refill in 20 minutes and 13 seconds.

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 3, 2026

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@demo/notebooks/03_langgraph_agent.ipynb`:
- Around line 30-42: Restore the git-pinned SDK spec used for Colab to avoid
proto mismatches: introduce a variable named _coordinode_spec that selects the
git+https URL with the specific commit SHA when IN_COLAB and "coordinode"
otherwise, replace the plain "coordinode" entry in the pkgs list with
_coordinode_spec, and ensure subprocess.run (the pip install call that consumes
pkgs) installs the _coordinode_spec and still prefixes with
"coordinode-embedded" when IN_COLAB as before.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 9030cff1-3976-4873-a014-7266377f89e7

📥 Commits

Reviewing files that changed from the base of the PR and between 9da6f48 and 5765484.

📒 Files selected for processing (5)
  • README.md
  • demo/notebooks/00_seed_data.ipynb
  • demo/notebooks/01_llama_index_property_graph.ipynb
  • demo/notebooks/02_langchain_graph_chain.ipynb
  • demo/notebooks/03_langgraph_agent.ipynb

Comment thread demo/notebooks/03_langgraph_agent.ipynb
@polaz polaz merged commit 834edfd into main May 3, 2026
8 checks passed
@polaz polaz deleted the fix/#63-pypi-install-notebooks branch May 3, 2026 15:45
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