Skip to content

docs: sync bilingual benchmark copy - #487

Merged
LivXue merged 9 commits into
mainfrom
docs/readme_followup
Sep 18, 2026
Merged

LivXue merged 9 commits into
mainfrom
docs/readme_followup

Conversation

@LivXue

@LivXue LivXue commented Sep 18, 2026

Copy link
Copy Markdown
Member

Summary

Synchronize the Chinese README with the updated English README. The change aligns the Raven introduction, benchmark captions, built-in agent copy, third-party agent positioning, and the removal of the obsolete benchmark summary section.

The change also adds the updated wording for data analysis, PowerPoint slide generation, and unattended AI4S benchmark captions while preserving all chart links and remaining bilingual sections.

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 run:

  • uv run --no-sync pytest tests/test_readme_scope_canon.py tests/test_cli_onboard_commands.py -k readme_quickstart -q (1 passed)
  • uv run --no-sync pytest tests/test_docker_runtime.py -k container_exposes_no_engine_selector -q (1 passed)
  • UV_NO_SYNC=1 make check-large-files check-source-language PYTHON_VERSION=.venv/bin/python COMMIT_RANGE=HEAD (passed)
  • git diff --check (passed)

Risk

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

This is a documentation-only change. Reverting the commit restores the previous README wording.

Related Issues

N/A

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

@gloryfromca gloryfromca left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No blockers; this can merge as far as I am concerned.

Reviewed the full github/main...HEAD diff, surrounding bilingual README content, relevant history, and the repository rules and context vocabulary. I verified the new transport and preset-count copy against the sub-agent preset implementation (13 presets across ACP/OpenAI-compatible APIs, with CLI still supported), and confirmed the obsolete benchmark block is removed symmetrically. This is documentation-only, so there is no runtime caller or backward-compatibility impact; no tests were changed or weakened.

Verification: both focused pytest selections passed (1 test each), the large-file and source-language gates passed when invoked directly because make is unavailable on this host, and git diff --check passed. All completed GitHub checks are green; one unit-test shard remains in progress at review time.

@LivXue

LivXue commented Sep 18, 2026

Copy link
Copy Markdown
Member Author

README overview image for the What is Raven section.

Raven one surface, all agents workflow

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

@gloryfromca gloryfromca left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No blockers; this can merge as far as I am concerned.

Reviewed the new revision delta and the complete PR diff. The overview asset resolves as a valid 2172x724 PNG, its workflow matches the surrounding Raven architecture copy, and it is inserted identically in both READMEs. Keeping the image externally hosted complies with the repository asset rule. The change has no runtime or backward-compatibility impact and does not alter tests.

Verification: both focused README-related pytest selections passed (1 test each), the large-file and source-language gates passed across github/main...HEAD, and git diff --check passed. All completed GitHub checks are green; three unit-test shards remain in progress at review time.

@LivXue

LivXue commented Sep 18, 2026

Copy link
Copy Markdown
Member Author

Updated README overview image with reduced vertical whitespace.

Raven one surface, all agents workflow

Co-authored-by: Claude (claude-opus-5[1m]) <noreply@anthropic.com>
@LivXue

LivXue commented Sep 18, 2026

Copy link
Copy Markdown
Member Author

Updated Chinese README overview image with reduced vertical whitespace.

Raven unified surface and agent workflow

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

@gloryfromca gloryfromca left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No blockers; suggestions only, and they are marked inline.

Reviewed the revision delta, the complete PR diff, both replacement image assets, repository rules, compatibility, and test integrity. The tightened images render correctly and the new captions are bilingual. One localization suggestion remains inline: the Chinese README points to the English-labeled image even though the PR supplies a Chinese-labeled counterpart. This does not block the fifth review round because the surrounding Chinese caption and prose remain usable.

Verification: both focused README-related pytest selections passed (1 test each), the large-file and source-language gates passed across github/main...HEAD, and git diff --check passed. CI shard 4 reported 5,549 passing tests but failed its strict idle monitor on an untouched PowerPoint test; that exact test passed locally with --idle-ceiling-strict (1 passed).

Comment thread README.zh-CN.md Resolved

@gloryfromca gloryfromca left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No blockers; this can merge as far as I am concerned.

Reviewed the one-line revision delta and the complete PR diff. The Chinese README now uses the localized image for both href and src, with matching alt text, so the prior localization follow-up is fully addressed. No runtime, compatibility, architecture, or test-integrity concern is introduced.

Verification: both focused README-related pytest selections passed (1 test each), the large-file and source-language gates passed across github/main...HEAD, and git diff --check passed. All current GitHub CI checks are green.

LivXue and others added 2 commits September 18, 2026 04:35
Co-authored-by: Claude (claude-opus-5[1m]) <noreply@anthropic.com>
Co-authored-by: Claude (claude-opus-5[1m]) <noreply@anthropic.com>

@gloryfromca gloryfromca left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No blockers; this can merge as far as I am concerned.

Reviewed the caption-only revision delta, the complete PR diff, relevant history, and the canonical DAG terminology in CONTEXT.md and the orchestration implementation. The claim matches Raven's run_subagent_dag architecture, and the English and Chinese captions remain semantically aligned. No runtime, compatibility, architecture, or test-integrity concern is introduced.

Verification: both focused README-related pytest selections passed (1 test each), the large-file and source-language gates passed across github/main...HEAD, and git diff --check passed. All completed GitHub checks are green; four unit-test shards remain in progress at review time.

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

@gloryfromca gloryfromca left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No blockers; this can merge as far as I am concerned.

Reviewed the one-line caption delta and the complete PR diff. The simplified English wording preserves the validated DAG-orchestration claim and remains semantically aligned with the Chinese caption. No runtime, compatibility, architecture, or test-integrity concern is introduced.

Verification: both focused README-related pytest selections passed (1 test each), the large-file and source-language gates passed across github/main...HEAD, and git diff --check passed. All completed GitHub checks are green; three unit-test shards remain in progress at review time.

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

@gloryfromca gloryfromca left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No blockers; this can merge as far as I am concerned.

Reviewed the one-line Chinese-caption delta and the complete PR diff. The revised lead phrase remains consistent with the English positioning and the surrounding multi-domain orchestration description. No runtime, compatibility, architecture, or test-integrity concern is introduced.

Verification: both focused README-related pytest selections passed (1 test each), the large-file and source-language gates passed across github/main...HEAD, and git diff --check passed. All completed GitHub checks are green; three unit-test shards remain in progress at review time.

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

@gloryfromca gloryfromca left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No blockers; this can merge as far as I am concerned.

Reviewed the two-line Chinese benchmark-caption delta and the complete PR diff. Both linked charts place Raven first (DataAgentBench: 0.8762; PresentBench: 80.2), so the more precise wording matches the existing English "tops" claims. I also covered repository rules, surrounding bilingual copy and history, compatibility, architecture, and test integrity; this remains documentation-only, with no runtime callers affected and no tests weakened.

Verification: both focused pytest selections passed (1 test each), the large-file and source-language gates passed across github/main...HEAD, and git diff --check passed. All completed GitHub checks are green; three unit-test shards remain in progress at review time.

@LivXue
LivXue requested a review from ZuyiZhou September 18, 2026 05:41

@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. Docs only, both halves move together, all checks green.

Checked:

  • Bilingual parity holds caption for caption; the same Benchmarks section and the same demo video are dropped from both files, and the new hero image is the localized pair.
  • The dropped section orphans nothing: no anchor anywhere links to it, and EvoAgentBench keeps its link in the ecosystem list (README.md:465).
  • "presets for 13 third-party agents" is exact: THIRD_PARTY_PRESET_NAMES holds 13 keys, and they are the 13 named in the image alt text.

One note, not blocking: the title says "sync bilingual benchmark copy", but this also deletes the whole Benchmarks section (the efficiency / self-evolution / proactivity table, its "results describe the published test configurations" caveat, and the demo video). That is a larger removal than the title advertises, and the caveat sentence was the only place qualifying those numbers. Fine if deliberate; worth saying so in the description.

@LivXue
LivXue merged commit c65c205 into main Sep 18, 2026
19 checks passed
@LivXue
LivXue deleted the docs/readme_followup branch September 18, 2026 05:53
@LivXue

LivXue commented Sep 20, 2026

Copy link
Copy Markdown
Member Author

Re-rendered README overview workflow images (English and Chinese).

Raven one surface, all agents workflow

Raven unified surface and agent workflow

@LivXue

LivXue commented Sep 20, 2026

Copy link
Copy Markdown
Member Author

Revised README overview workflow images (English and Chinese). These supersede the pair in the previous comment; the READMEs reference this set.

Raven one surface, all agents workflow

Raven unified surface and agent workflow

LivXue added a commit that referenced this pull request Oct 12, 2026
## Summary

Synchronize the Chinese README with the updated English README. The
change aligns the Raven introduction, benchmark captions, built-in agent
copy, third-party agent positioning, and the removal of the obsolete
benchmark summary section.

The change also adds the updated wording for data analysis, PowerPoint
slide generation, and unattended AI4S benchmark captions while
preserving all chart links and remaining bilingual sections.

## 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 run:

- `uv run --no-sync pytest tests/test_readme_scope_canon.py
tests/test_cli_onboard_commands.py -k readme_quickstart -q` (1 passed)
- `uv run --no-sync pytest tests/test_docker_runtime.py -k
container_exposes_no_engine_selector -q` (1 passed)
- `UV_NO_SYNC=1 make check-large-files check-source-language
PYTHON_VERSION=.venv/bin/python COMMIT_RANGE=HEAD` (passed)
- `git diff --check` (passed)

## Risk

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

This is a documentation-only change. Reverting the commit restores the
previous README wording.

## Related Issues

N/A

---------

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.

3 participants