Skip to content

Remove staled cli arg in vlmevalkit docs#4598

Merged
lvhan028 merged 1 commit into
InternLM:mainfrom
CUHKSZzxy:fix-doc
May 19, 2026
Merged

Remove staled cli arg in vlmevalkit docs#4598
lvhan028 merged 1 commit into
InternLM:mainfrom
CUHKSZzxy:fix-doc

Conversation

@CUHKSZzxy
Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings May 19, 2026 04:34
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the VLMEvalKit evaluation documentation to remove a stale run.py CLI argument that is no longer needed for LMDeploy-backed API evaluation.

Changes:

  • Removed --api 123 from the example python run.py ... command in both English and Chinese docs.
  • Kept parameter explanations consistent (no remaining references to --api beyond --api-nproc).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
docs/zh_cn/benchmark/evaluate_with_vlmevalkit.md Removes the stale --api 123 argument from the OCRBench run command example.
docs/en/benchmark/evaluate_with_vlmevalkit.md Removes the stale --api 123 argument from the OCRBench run command example.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@lvhan028 lvhan028 merged commit 7a0c3df into InternLM:main May 19, 2026
7 checks passed
@CUHKSZzxy CUHKSZzxy deleted the fix-doc branch May 20, 2026 02:40
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