Skip to content

Add vlmbench-cli-skill with SKILL.md and MODELS.md - #6

Closed
devin-ai-integration[bot] wants to merge 2 commits into
mainfrom
devin/1773239996-add-vlmbench-cli-skill
Closed

devin-ai-integration[bot] wants to merge 2 commits into
mainfrom
devin/1773239996-add-vlmbench-cli-skill

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Mar 11, 2026

Copy link
Copy Markdown
Contributor

Summary

Copies the vlmbench CLI skill files from vlm-run/vlmbench/.claude/skills/vlmbench/ into a new skills/vlmbench-cli-skill/ directory in this repo.

Files added:

  • skills/vlmbench-cli-skill/SKILL.md — CLI usage, backend resolution, concurrency, metrics, GPU detection
  • skills/vlmbench-cli-skill/MODELS.md — Supported models table with vLLM serve-args and model profiles

Updates since last revision

  • Updated MODELS.md to use uvx vlmbench instead of bare vlmbench in CLI examples (per review feedback). This means the files are no longer exact verbatim copies from the source repo — SKILL.md already used uvx vlmbench, but MODELS.md had two bare vlmbench invocations that have been corrected.

Review & Testing Checklist for Human

  • Filename check: The source repo has SKILL.md (singular), not SKILLS.md as mentioned in the original request. Verify this is the intended name, or rename to SKILLS.md if preferred.
  • Confirm the content of both files is up to date with the latest version in vlm-run/vlmbench (files were copied at the time of this PR and may drift).

Notes

Co-Authored-By: Sudeep Pillai <sudeep.pillai@gmail.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Comment thread skills/vlmbench-cli-skill/MODELS.md Outdated
Default vLLM serving arguments for OCR/VLM models. Use `--serve-args` to pass these when running benchmarks:

```bash
vlmbench run --serve --backend vllm --model <model-id> --serve-args '<args>' -i <input>

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.

Use uvx vlmbench instead of vlmbench across all the README

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated — replaced vlmbench with uvx vlmbench in the CLI examples in MODELS.md (lines 6 and 25). SKILL.md already used uvx vlmbench throughout.

Co-Authored-By: Sudeep Pillai <sudeep.pillai@gmail.com>
spillai pushed a commit that referenced this pull request Sep 11, 2026
PR #6 proposed the same skill and uniquely carried a model -> serve-args
table. Fold a corrected version into SKILL.md so #6 can be closed without
losing content, keeping the skill a single self-contained file.

Two things in #6's copy were stale against vlmbench main and are fixed here:

- Profiles are flat `vlmbench/profiles/<name>.yaml` files, not per-model
  directories holding `config.toml` + `setup.sh`.
- Its profile list named 4 profiles; the package ships 10. Point at
  `uvx vlmbench profiles` instead of a table that drifts.

Also drop #6's `make build/serve/benchmark` block: those targets need a
clone of the vlmbench repo and do not apply to the uvx workflow this
skill documents.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UY3EWR8JezM8uBfUHv6Z7V

spillai commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Closing as superseded by #13, which adds the same skill at the same path (skills/vlmbench-cli-skill/). Both PRs create SKILL.md there, so only one can land.

Carried over from this PR: the model → --serve-args table, which was the one thing #13 lacked. It now lives inline in SKILL.md rather than in a separate MODELS.md, keeping the skill one self-contained file.

Two things in this PR's copy were stale against vlm-run/vlmbench@main and were corrected on the way over:

This PR says vlmbench main
Profile storage directories vlmbench/profiles/<name>/ with config.toml + setup.sh flat vlmbench/profiles/<name>.yaml; no config.toml or setup.sh exists
Profile inventory 4 profiles 10 profiles ship in the package

#13 points at uvx vlmbench profiles for the live list instead of a static table that drifts. The make build/serve/benchmark block was dropped, since those targets need a clone of the vlmbench repo and do not apply to the uvx workflow the skill documents.

Also in #13 and not here: registration in .claude-plugin/marketplace.json, without which /plugin install vlmbench-cli-skill@vlm-run/skills does not resolve; a row in the README skills table; and a frontmatter name matching the directory. Its body is written for the current CLI, which this snapshot predates: results now default to ~/.vlmbench/benchmarks/ rather than ./results/, --model is optional when --base-url is set, --serve must be passed explicitly to auto-start a server, and text-only dataset runs are supported.

The uvx vlmbench review note on this PR from @spillai is satisfied in #13, where every example uses uvx.


Generated by Claude Code

@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

❌ Cannot revive Devin session - the session is too old. Please start a new session instead.

View session

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.

1 participant