Repository navigation
Say milestones 0 to 4 are complete and the DocC site is published (README, docs) - #138
Merged
Merged
Conversation
Milestone 4's tracking issue #42 and its work issues #43 to #48 are closed, so the README's Status says milestones 0 to 4 are complete instead of milestone 4 in progress. The Verdict, Laya, JevBench comparison, calibration report and JevK5 facts keep their numbers. The "Not there yet" list is unchanged: #50 to #53, #59 and #65 are open, and the generation pull request for #50 to #52 has not merged. GitHub Pages serves the site the Documentation workflow deploys from main, so the README, docs/README.md and docs/09-conformance-and-testing.md say the workflow publishes it there instead of "once GitHub Pages is enabled". D-047 keeps its wording: a decision records what was true when it was written.
Contributor
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The documentation-only updates match the completed milestone and successful main-branch documentation deployment.
Review effort: Balanced
Findings: None
What changed in this PR
Updates project status and documentation links to reflect milestone 4’s completion and the active DocC site.
Changes:
- Marks milestones 0–4 complete.
- Removes obsolete GitHub Pages enablement caveats.
- Adds the published DocC URL to conformance documentation.
| File | Description |
|---|---|
README.md |
Updates milestone and documentation status. |
docs/README.md |
Reflects active DocC publishing. |
docs/09-conformance-and-testing.md |
Records the published documentation URL. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
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.
Docs only. Brings the README's status lines, and the two docs that repeat the GitHub Pages line, up to date with main before the project is advertised. No Swift, fixture or workflow file changes.
Every state below was read on 2026-10-06 with
ghp issue view N --repo Algorythm-Canada/OpenJevSwift --json state,title,ghp pr listandcurl, once before editing and again right before opening this pull request.The stale lines
1. README.md, Status (line 72)
steps,samplesandsequentialare verified end to end [...]"openjevtool, and the read extensions and images." The rest of the paragraph keeps its facts and numbers:steps,samplesandsequential(steps (1 to 8) end to end on the DiffusionGemma backend #43 to sequential mode end to end: prefix continuation and earlier answers in the prompt #45), images bit for bit on the oracle's kernels (Gemma 4 image preprocessing parity for DiffusionGemma prompts #46 to images request field end to end #48), Verdict and Laya, the JevBench comparison and the calibration report (JevBench harness and agreement report against upstream OpenJev #61 and Calibration report for DiffusionGemma reads (and decision on optional temperature scaling) #62), and JevK5 (JevK5 letter-readout backend on Qwen3.5 via MLXLLM, with checkpoint conversion #55) with 230 of JevBench's 231 items (D-052). The only number that changes is the milestone range, 3 to 4.2. README.md, "Not there yet" (line 81): checked, unchanged
issue-50-51-52-generation-and-think), which has not merged, sothinkand chat stay as main has them, matching docs/compatibility.md on main, whosemlxrow has "no, issue The think option: thought generation before a read, prefix continuation, billing #52" underthinkand "the route yes; the model no, issue Block generation loop: canvas sizing, cache commits, streaming detokenizer, stop ids #51" under Chat.3. README.md, Documentation (line 112)
curl -s -o /dev/null -w '%{http_code}' https://algorythm-canada.github.io/OpenJevSwift/prints200. The Pages API (ghp api repos/Algorythm-Canada/OpenJevSwift/pages) reportsbuild_type: workflow. The Documentation run for d25223a (run 37486332329) ranDeploy to GitHub Pagessuccessfully at 15:30 UTC; it is one of 15github-pagesdeployments since 2026-10-02.4. docs/09-conformance-and-testing.md (line 367)
mainonce GitHub Pages is enabled."mainto https://algorythm-canada.github.io/OpenJevSwift/." The rest of the bullet is only rewrapped.5. docs/README.md (line 34), not in the original list
docs/06-decisions.md is left alone: D-047 records what was true when it was written.
Found, not changed here
Tools/docs/build-site.shbuilds five modules, but docs/09-conformance-and-testing.md:365 (the bullet edited in 4), docs/development.md:357, 385 and 618, the comment at .github/workflows/docs.yml:70, and Tools/docs/index.html:93 still say four. The last one is on the live site's front page, as "All four modules in one index" under a list of five. This pull request keeps every number as it is, and changing the front page redeploys the site, so that fix belongs in a separate change.thinkand chat are served onmlx, and the README's "Not there yet" lines for them go stale. If Generate text, think and chat on the MLX backend: the diffusion sampler, the block loop, think and the chat model (#50, #51, #52, #53, D-059) #137 merges first, this pull request needs one more commit before it merges.Checks
make lintpasses.swift testwas not run.