Skip to content

Add model cards for Seed2.0 and DeepSeek V4.1 (issue #607) - #610

Open
jellyfishing2346 wants to merge 2 commits into
ktwu01:mainfrom
jellyfishing2346:feat/add-model-cards
Open

jellyfishing2346 wants to merge 2 commits into
ktwu01:mainfrom
jellyfishing2346:feat/add-model-cards

Conversation

@jellyfishing2346

Copy link
Copy Markdown

Summary

  • Add ByteDance Seed2.0 model card (25 benchmarks)
  • Add DeepSeek V4.1-Flash technical report (16 benchmarks)
  • Add 15 missing benchmark definitions (BeyondAIME, PhyBench, BABE, KORBench, etc.)

Impact

  • Increases model_reports source count from 53 to 55
  • Adds 2 new models to the registry: Seed2.0 (Pro/Lite/Mini) and DeepSeek-V4.1-Flash
  • Expands benchmark coverage with instruments previously absent from the catalog
  • All 1,334 tests pass after full pipeline rebuild

Test plan

  • Added model card entries follow registry conventions
  • All benchmark IDs resolve to existing definitions
  • Ran benchmark-radar normalize-catalog → 1,298 records
  • Ran benchmark-radar classify → 872 models
  • Ran benchmark-radar build-data-release → CLI bundle updated
  • Rebuilt logo registry to include new models
  • All 1,334 tests pass

Closes #607

   Seed2.0 reports 25 benchmarks across reasoning, math, science, coding,
   and agent capabilities. DeepSeek V4.1-Flash reports 16 benchmarks with
   focus on agentic tasks and KV cache compression architecture.

   Also adds 15 benchmark definitions referenced by these documents but
   previously missing from the registry (BeyondAIME, PhyBench, BABE, etc.).
@ktwu01

ktwu01 commented Sep 12, 2026

Copy link
Copy Markdown
Owner

Thanks for this. The mentions are the right first step, but let's not stop at cards plus benchmark definitions: please add the scores too, in the existing format.

Where scores live

model_cards.yml deliberately stores a mention, not a number (see its header). Scores go in a separate file, data/benchmark_scores.yml, which already has 4,030 lines of precedent. It has two blocks you'd need to touch:

1. benchmarks: (metric identity). None of the 15 benchmark ids this PR adds have an entry yet, so a score row for them has no metric, direction or unit to render against:

  - benchmark_id: beyond_aime
    metric: accuracy
    direction: higher_is_better
    unit: percent

direction matters: it exists because not every metric improves upward, so please set it from what the benchmark actually reports rather than defaulting it.

2. results: (one row per number). The nearest template is the existing DeepSeek-V4-Pro block around data/benchmark_scores.yml:2153, which is exactly the shape the DeepSeek-V4.1-Flash rows should follow:

  - benchmark_id: gpqa_diamond
    instrument: gpqa_diamond
    protocol: think max, pass@1
    model: DeepSeek-V4.1-Flash
    organization: DeepSeek
    source_id: deepseek_v4_1_flash_technical_report
    reported_at: 2026-09-10
    value: 88.1
    read_from: pdf_text

Three fields carry most of the weight, so please don't shortcut them:

  • instrument is the version identity, separate from benchmark_id. If the Seed2.0 card reports LiveCodeBench v5 or Terminal-Bench 2.1, that belongs here, so a task-set change never reads as model progress.
  • protocol is the comparability class. Record whatever the source said moves the number (thinking budget, tool access, pass@1 vs consensus@k). Silence is not agreement, so don't leave it generic to make rows look joinable.
  • source_id must be the card id you registered in this PR (bytedance_seed2_0_model_card, deepseek_v4_1_flash_technical_report), and read_from should be pdf_text for both, since both sources are PDFs.

Seed2.0 ships Pro / Lite / Mini. The card entry can stay one row, but score rows are per model, so please split model: per variant instead of collapsing three systems into one label.

Per the file's own rule: read each number out of the cited document and record it exactly as printed. If a table is an image and a value isn't certain, add nothing. An absent row is honest, a guessed row isn't.

Blocker on the benchmark URLs

Separately, and this needs fixing before merge: most of the new benchmark url values don't resolve. I checked 12 of them and 10 return 404:

url status
github.com/microsoft/ProMath (beyond_aime) 404
github.com/PhyBench/PhyBench 404
github.com/babe-eval/babe 404
github.com/korbench/korbench 404
github.com/mars-bench/mars-bench 404
github.com/multi-challenge/multi-challenge 404
github.com/collie-bench/collie 404
github.com/lpfqa/lpfqa 404
github.com/swe-lancer/swe-lancer 404
github.com/exploitgym/exploitgym 404
github.com/sec-bench/sec-bench 200
github.com/zerobench/zerobench 200

Also, inverse_ifeval points at the original google-research IFEval directory, which is a different benchmark from Inverse IFEval.

These read as plausible-looking guesses rather than verified links, and that's a problem in a file whose whole contract is that any row can be checked against its source. The two card URLs are fine (both the Seed2.0 PDF and the DeepSeek V4.1 report return 200), so the fix is to pull each benchmark's real homepage or paper from the citation in the card itself, and drop url where you can't find one. Same goes for the caveat text: it should reflect what the benchmark actually measures, not a generic gloss inferred from the name (the current BABE and COLLIE descriptions look guessed).

Once the urls are verified and the score rows are in, this is a solid addition.

   - Add 35 score rows for Seed2.0 Pro/Lite/Mini and 3 for DeepSeek-V4.1-Flash from source PDFs
   - Add metric definitions for 14 new benchmarks
   - Fix 13 dead/incorrect benchmark URLs with verified sources
   - Update caveat text for BABE, COLLIE, inverse_ifeval
   - Split Seed2.0 scores by model variant as required
@jellyfishing2346

Copy link
Copy Markdown
Author

Thank you for the feedback, I took the time to look over your comments. Let me know if there is anything else I need to improve upon.

@ktwu01 ktwu01 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks for following up on the earlier feedback and for correcting the benchmark URLs. I re-ran the required CI sequence from a clean worktree, and there are still a few blockers before this can merge:

  1. benchmark-radar normalize-catalog fails immediately because the existing FrontierChallenge leaderboard was moved under model_cards:. It is then parsed as a model card and is missing organization and model. Please preserve the original top-level source_documents: block and its FrontierChallenge entry. The new trailing source_documents: entry is also incomplete (name, publisher, and benchmarks are required) and appears unnecessary for registering Inverse IFEval as a benchmark.

  2. The 35 new score rows were inserted before the existing results: key, so they are currently part of the metric-definition benchmarks: list. As a result, none of them load as score results. Please move them under the existing results: block.

  3. Score coverage is still incomplete. The added rows cover only 12 of the 26 Seed2.0 benchmark IDs and 3 of the 16 DeepSeek benchmark IDs listed by the cards. Per docs/sop-add-model-cards.md, please add every numeric score that can be read with certainty from both cited PDFs, including scores for benchmarks whose metric definitions already existed.

  4. Please reconcile metric identity with the reported protocol. In particular, ZeroBench is declared as pass@1, while the DeepSeek row is explicitly Pass@5, with tools; those cannot share one metric identity as written.

  5. Please restore the unrelated brokenarxiv entry removed from the Tencent Hy4 card. After correcting the YAML section placement in a disposable worktree, the pipeline generated 875 models rather than the committed 872, and the suite finished with 1,332 passed / 2 failed because the three Seed2.0 score variants were missing from the logo registry. Please regenerate the tracked model and logo artifacts after the source data is corrected, then run all six CI commands from AGENTS.md in a fresh worktree.

The corrected benchmark URLs all resolve successfully, so that part looks good. Thank you again for working through this data-heavy addition; once the section placement, score completeness, metric consistency, and generated artifacts are fixed, I will be happy to re-review.

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.

[6 points] add model cards

2 participants