Skip to content

Move to uv and update no-key math demo #43

Merged
bordeauxred merged 5 commits into
mainfrom
enhancement/issue-31-ship-installed-no-key-math-demo-and-verify-it-from-the-wheel
Apr 19, 2026
Merged

Move to uv and update no-key math demo #43
bordeauxred merged 5 commits into
mainfrom
enhancement/issue-31-ship-installed-no-key-math-demo-and-verify-it-from-the-wheel

Conversation

@tianyibigdata
Copy link
Copy Markdown
Collaborator

@tianyibigdata tianyibigdata commented Apr 19, 2026

Improves the usability of the clawloop demo experience after installation by making the demo accessible both from the installed package and from source clones.

@tianyibigdata tianyibigdata self-assigned this Apr 19, 2026
@tianyibigdata tianyibigdata linked an issue Apr 19, 2026 that may be closed by this pull request
6 tasks
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a built-in math learning loop demo, accessible via a new demo subcommand in the CLI and as a package-resident module. The project now requires Python 3.12+, with updated documentation and installation instructions centered around the uv package manager. The existing math demo was refactored into a core module with a thin shim maintained in the examples directory for backward compatibility. Review feedback suggests refining configuration logic to support zero values, optimizing I/O by utilizing in-memory results, and improving test portability by replacing hardcoded paths with pytest fixtures.

Comment thread clawloop/demo_math.py Outdated
Comment thread clawloop/demo_math.py
Comment thread tests/test_demo_math.py Outdated
Signed-off-by: getianyi <getianyi602@gmail.com>
@tianyibigdata tianyibigdata force-pushed the enhancement/issue-31-ship-installed-no-key-math-demo-and-verify-it-from-the-wheel branch from 57bc0a8 to 61d43f5 Compare April 19, 2026 15:16
Comment thread README.md Outdated
| You have... | Start here | What it shows |
|---|---|---|
| A Python agent | [`examples/demo_math.py`](examples/demo_math.py) | Full learning loop with `ClawLoopAgent` |
| A repo clone | `uv run clawloop demo math --dry-run` | Full learning loop, no API keys needed |
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we have to update the other demos to uv?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yes — addressed in c46be61. All python examples/*.py commands in README.md, examples/README.md, docs/getting-started.md, and docs/index.md now use uv run. Grep confirms no plain-python commands remain in user-facing docs.

@bordeauxred
Copy link
Copy Markdown
Contributor

thank you @tianyibigdata, lgtm. Please add initial description

@bordeauxred bordeauxred changed the title Ship installed no-key math demo and verify it from the wheel Move to uv and update no-key math demo Apr 19, 2026
@bordeauxred bordeauxred merged commit 598d5d2 into main Apr 19, 2026
6 checks passed
@bordeauxred bordeauxred deleted the enhancement/issue-31-ship-installed-no-key-math-demo-and-verify-it-from-the-wheel branch April 19, 2026 16:16
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.

Ship installed no-key math demo and verify it from the wheel

2 participants