Repository navigation
Phase 6: quality + docs - #6
Merged
Merged
Conversation
- tests/test_geometry.py: cross-cutting Hypothesis geometry properties — every step turns the heading by exactly the turn angle, right-angle walk segments are perpendicular, the 3-D corkscrew's xy projection equals the 2-D walk, and the spiral radius strictly increases. - PRE-MORTEM.md (failure modes + guards), MUTATION-TESTING.md (mutmut scope), CHANGES_SUMMARY.md (open decisions: fonts, walk overlap, recurrence cost, HTML tier, integrations). - docs: the spine tutorial (the four steps + the data contract); README quickstart shows every preset and a roadmap; quickstart "more presets". - text_walk: drop the no-op tight_layout (axes are off) that emitted a spurious warning. mutmut selection now includes walk3d + geometry tests. make ci green (184 tests, 1 opt-in skip, no warnings); mkdocs --strict green. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Import sentence-transformers by name via importlib.import_module instead of a static import with a ty-ignore. The previous ignore was used locally (extra absent) but would be unused in CI (extra installed via --all-extras), which a strict checker can flag. The string import is neither required to import the module nor statically type-checked in either environment. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
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.
Phase 6 — quality + docs
The final pass: deeper property tests and the family's quality docs. Stacked on #5.
Tests
tests/test_geometry.py— cross-cutting Hypothesis invariants beyond the per-module tests:z_step;Docs
PRE-MORTEM.md— failure modes and their guards (segmentation heuristics, walk self-overlap, outline-vs-advance width, Jaccard noise, O(N²) recurrence, model download, headless rendering, unvendored fonts).MUTATION-TESTING.md— mutmut scope (the layout geometry) and how to run it.CHANGES_SUMMARY.md— open decisions (fonts, one-way turn, width metric, recurrence cost, HTML tier, integrations).Fixes
text_walkdrops a no-optight_layout(axes are off) that emitted a spurious warning — the suite is now warning-free.Verification
make cigreen: ruff + ty + 184 tests / 1 opt-in skip, no warnings.mkdocs build --strictgreen.This completes the planned build (Phases 0–6). The five presets, the optional
[graph]analysis layer, and the docs are all in. Remaining stretch items (vendored fonts,[kenon]/[chronowords]integrations, HTML/WebGL export) are tracked inCHANGES_SUMMARY.md.🤖 Generated with Claude Code