Skip to content

docs: refresh AI4AI benchmark chart - #498

Merged
LivXue merged 1 commit into
mainfrom
docs/update_ai4ai_chart
Sep 18, 2026
Merged

LivXue merged 1 commit into
mainfrom
docs/update_ai4ai_chart

Conversation

@LivXue

@LivXue LivXue commented Sep 18, 2026

Copy link
Copy Markdown
Member

Summary

Update the hosted AI4AI (Nanochat 50M Pretraining) chart reference in both bilingual READMEs.

  • Replace the old image attachment URL in README.md and README.zh-CN.md.
  • Use the updated chart with the Bits Per Byte (BPB) label and the Lower Is Better annotation above the BPB values.
  • Keep the chart hosted as a GitHub attachment; no image binary is added to the repository.

Type

  • Fix
  • Feature
  • Docs
  • CI / tooling
  • Refactor
  • Other

Verification

  • Relevant tests pass locally
  • Relevant lint / type checks pass locally
  • User-facing docs or screenshots are updated when needed

Commands:

  • git diff --check
  • UV_NO_SYNC=1 make check-large-files check-source-language PYTHON_VERSION=.venv/bin/python COMMIT_RANGE=HEAD
  • UV_NO_SYNC=1 uv run --no-sync pytest tests/test_readme_scope_canon.py tests/test_cli_onboard_commands.py -k readme_quickstart -q
  • Downloaded attachment SHA-256 matches the local PNG.

Risk

  • Security impact considered
  • Backward compatibility considered
  • Rollback path is clear for risky changes

This is a documentation-only URL update. Reverting the commit restores the previous chart.

Related Issues

#465

Update the hosted AI4AI chart reference in both bilingual READMEs.

Co-authored-by: Claude (claude-opus-5[1m]) <noreply@anthropic.com>
@LivXue
LivXue requested a review from ZuyiZhou September 18, 2026 08:05

@ZuyiZhou ZuyiZhou left a comment

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.

Approving. Two lines, one asset id, the same swap in both files.

I fetched both PNGs and compared them: every number is identical -- BPB 1.053 / 1.038 / 1.037, runtime 4.99 / 4.90 / 3.74, tokens 13.35 / 9.38 / 3.79, cost $50.00 / $39.40 / $0.97, the same three series and the same 2x A800 note. So this is a cosmetic refresh, not a new measurement, and nothing in the surrounding prose needed to move with it. The BPB panel now carries its unit in the heading (Bits Per Byte (BPB)) instead of on a line below, which is what the alt text already said, and it gains a "lower is better" arrow.

One nit, not worth a round trip: "Lower Is Better" now appears twice, top right for the figure and again inside the BPB panel. It is true of all four panels, so the per-panel one reads as if it were not.

The red check is not this PR. unit (py3.12 / 1/4) reports 5489 passed, 9 skipped; what failed the job is --idle-ceiling-strict tripping on tests/test_ppt_engine_assets_script_helpers.py::test_a_formula_that_stacks_is_typeset_as_a_picture_in_the_decks_ink at 5.76s idle -- a runner-speed flake in the deck engine that a two-line README edit cannot reach. main's last two runs are green. Worth a re-run before merging rather than a change here.

@LivXue
LivXue merged commit 027637b into main Sep 18, 2026
17 of 18 checks passed
@LivXue
LivXue deleted the docs/update_ai4ai_chart branch September 18, 2026 08:09
LivXue added a commit that referenced this pull request Oct 12, 2026
## Summary

Update the hosted AI4AI (Nanochat 50M Pretraining) chart reference in
both bilingual READMEs.

- Replace the old image attachment URL in README.md and README.zh-CN.md.
- Use the updated chart with the Bits Per Byte (BPB) label and the Lower
Is Better annotation above the BPB values.
- Keep the chart hosted as a GitHub attachment; no image binary is added
to the repository.

## Type

- [ ] Fix
- [ ] Feature
- [x] Docs
- [ ] CI / tooling
- [ ] Refactor
- [ ] Other

## Verification

- [x] Relevant tests pass locally
- [x] Relevant lint / type checks pass locally
- [x] User-facing docs or screenshots are updated when needed

Commands:

- `git diff --check`
- `UV_NO_SYNC=1 make check-large-files check-source-language
PYTHON_VERSION=.venv/bin/python COMMIT_RANGE=HEAD`
- `UV_NO_SYNC=1 uv run --no-sync pytest tests/test_readme_scope_canon.py
tests/test_cli_onboard_commands.py -k readme_quickstart -q`
- Downloaded attachment SHA-256 matches the local PNG.

## Risk

- [x] Security impact considered
- [x] Backward compatibility considered
- [x] Rollback path is clear for risky changes

This is a documentation-only URL update. Reverting the commit restores
the previous chart.

## Related Issues

#465

Co-authored-by: Claude (claude-opus-5[1m]) <noreply@anthropic.com>
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.

2 participants