Repository navigation
docs(vmaf): research digest + ADR-1122 for VMAF v1 model port - #1082
Conversation
|
Apologies for the long silence on this — the repository had a CI deadlock that blocked every PR (including our own) from 2026-06-29 until today. Nothing to do with your work. Details in #1127 if you're curious: the Required Checks Aggregator treats a sibling job still Two things I've done here: 1. CI is now running on this PR for the first time. Fork PRs need a maintainer to approve workflow runs, and that never happened. Approved now. 2. Pushed one formatting-only commit (4256164) to get Heads-up if you ever reach for it: On the ADR numbers. You claimed ADR-1121 (in #1081) and ADR-1122 here, and those claims are correct and stand — I'm not going to renumber your work. But our allocator ( Also: several of the The v1 model research is genuinely good work — the feature-delta table and the |
Netflix shipped VMAF v1 in libvmaf v3.2.0 (June 2026): still a libsvm nu-SVR fusion, but VIF is removed and CAMBI (banding) + speed_chroma (chroma) become core features; ADM/motion move to adm3/motion3. Six built-in models for 1080p@3H / phone@5H / 4K@1.5H / 4K@3H (last on a [0,110] range) + HFR variants; Netflix recommends 10-bit measurement even for 8-bit SDR. Adds a cited research digest and ADR-1122 deciding to adopt v1 as an opt-in generation (v0.6.1 stays default), with the port scope: build in the JSONs, fix SYCL cambi/speed_chroma (now core, currently failing on Arc A380), handle the [0,110] range end-to-end, plumb CAMBI enc params, and add new v1 golden refs in separate tests (v0.6.1 golden pairs untouched). Aligns with the zero-copy 10-bit work (ADR-1121, PR VMAFx#1081). Refs ADR-1122. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01YCP4JCoduujJgccDRGvNKv
…R-1122 Formatting only — no content or conclusions changed. - table delimiter rows to the repo's "compact" pipe style (MD060) - `text` language on the feature_names fence (MD040) - backtick the viewing-condition identifiers `1080p@3H` / `phone@5H` / `4K@1.5H` / `4K@3H`; MD034 read the dot in `4K@1.5H` as a TLD and flagged it as a bare URL. Backticks are also more correct — these are technical identifiers, not prose. - angle-bracket the maintainer e-mail (MD034) markdownlint-cli2 over the PR's changed files now reports 0 issues. Note for anyone repeating this: `markdownlint-cli2 --fix` is unsafe here per ADR-0864. It rewrote `4K@1.5H` to `<4K@1.5H>`, silently corrupting the text. The autofix output was reverted and the fixes above applied by hand. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
4256164 to
f5ba7c9
Compare
Summary
Netflix shipped VMAF v1 in libvmaf
v3.2.0(June 2026). This PR adds a citedresearch digest and ADR-1122 deciding how the fork adopts it: opt-in v1
generation,
vmaf_v0.6.1stays default. Docs-only (planning) — no code yet.Key finding: v1 is still a libsvm ν-SVR fusion, but VIF is removed and CAMBI
(banding) +
speed_chroma(chroma) become core features (ADM/motion → adm3/motion3).Six built-in models (1080p@3H, phone@5H, 4K@1.5H, 4K@3H — last on [0, 110]) + HFR
variants; Netflix recommends 10-bit even for 8-bit SDR (aligns with ADR-1121 / PR #1081).
Type
docs— documentation onlyChecklist
docs/adr/_index_fragments/1122-*.md+_order.txt(README regenerated).Bug-status hygiene (ADR-0165)
no state delta: planning ADR, not a bug open/close/rule-out.Deep-dive deliverables (ADR-0108)
docs/research/2026-07-01-vmaf-v1-models.md.## Alternatives considered(opt-in vs v1-default vs don't-port vs models-only).AGENTS.mdinvariant note — no rebase-sensitive invariants: docs-only planning ADR, no code touched.Known follow-ups
The v1 implementation (build in JSONs, fix SYCL
cambi/speed_chroma,[0,110]handling, CAMBI enc-params, v1 golden tests, upstream sync to v3.2.0) is scoped in
ADR-1122 and tracked as a follow-up port — this PR only lands the decision + research.
🤖 Generated with Claude Code
https://claude.ai/code/session_01YCP4JCoduujJgccDRGvNKv