docs: fix inaccurate README/docs claims (verb count, dead links, stale what's-new, bad example flags)#243
Open
rylinjames wants to merge 1 commit into
Open
docs: fix inaccurate README/docs claims (verb count, dead links, stale what's-new, bad example flags)#243rylinjames wants to merge 1 commit into
rylinjames wants to merge 1 commit into
Conversation
…e what's-new, bad example flags) - README.md line 16: "eleven verbs" -> "a focused set of verbs" (code has 18+ visible top-level commands; the hardcoded count was wrong and brittle) - README.md line 275: remove hardcoded "11 visible verbs" count for same reason - README.md lines 14, 116, 484: de-link three dead hyperlinks to reflex_context/ (private vault not in this repo); keep surrounding prose, replace with inline backtick text noting the vault location - README.md: add v0.12.0 "What's new" entry (Reflex->Tether rename, fastcrest-tether PyPI dist, reflex compat shim through v0.13.x, removed v0.14.0) - CONTRIBUTING.md: replace nonexistent tether serve --policy smolvla --device cuda (--policy flag does not exist on serve; serve takes a positional export_dir) with tether go --model smolvla-base which is the correct one-command smoke test - examples/02-deploy-smolvla-jetson.md line 65: fix tether export --model smolvla-base --out ./smolvla-export/ to tether export smolvla-base --output ./smolvla-export/ (model is a positional arg, not --model; --out does not exist, correct flag is --output) - THIRD_PARTY_LICENSES.md: update "Reflex VLA" -> "Tether" in project attribution header; update src/reflex/ -> src/tether/ path references for this project's own vendored files (upstream MIT license text and vendor name unchanged) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Collaborator
Author
|
Reviewed the docs accuracy changes against the current CLI surface. Verified locally:
I cannot approve this PR through GitHub because it is authored by the same account, but the review is clean. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
@app.command()+ 12 non-hiddenapp.add_typer()groups). Rephrased both to "a focused set of verbs" / "Core user-facing verbs shown above" to avoid a brittle exact count.reflex_context/hyperlinks removed: Three clickable markdown links in README pointed into the privatereflex_context/vault (not in this repo):reflex_context/measured_numbers.md(×2) andreflex_context/03_experiments/2026-04-29-v07-runtime-spike.md. Each link is de-linked; surrounding prose is kept, with a note that it is in the private vault.fastcrest-tetherPyPI dist,tetherCLI/import, and thereflexcompat shim lifecycle (supported through v0.13.x, removed in v0.14.0). Mirrors CHANGELOG.mdd444db8.tether serve --policy smolvla --device cuda. The--policyflag does not exist ontether serve(which takes a positionalexport_dir). Replaced withtether go --model smolvla-base, the correct one-command smoke-test path.examples/02-deploy-smolvla-jetson.mdline 65):tether export --model smolvla-base --out ./smolvla-export/used two nonexistent flags. The realexportsignature hasMODELas a positional argument and--output(not--out). Fixed totether export smolvla-base --output ./smolvla-export/.src/reflex/paths describing where this project's own vendored code was lifted into were updated tosrc/tether/. Upstream third-party names, licenses, and the MIT license text are untouched.Before / after (key changes)
[reflex_context/measured_numbers.md](...)`reflex_context/measured_numbers.md` (private vault, not in this repo)[`reflex_context/03_experiments/...`](...)`reflex_context/03_experiments/...` (private vault, not in this repo)tether serve --policy smolvla --device cudatether go --model smolvla-basetether export --model smolvla-base --out ./smolvla-export/tether export smolvla-base --output ./smolvla-export/src/reflex/pathssrc/tether/pathsTest plan
tether export --helpshowsMODELas positional and--output(not--out) in the real CLItether serve --helpshows no--policyflag🤖 Generated with Claude Code