Skip to content

docs(readme): restore the badge set, add a banner, and bring back the two lost sections - #1508

Merged
lusoris merged 6 commits into
masterfrom
docs/readme-badges-and-polish
Sep 20, 2026
Merged

lusoris merged 6 commits into
masterfrom
docs/readme-badges-and-polish

Conversation

@lusoris

@lusoris lusoris commented Sep 20, 2026 •

Copy link
Copy Markdown
Contributor

Summary

The rc1 documentation pass (3cd0f0a, 2026-09-08) cut the README from 22 badges to four and dropped two sections with it. What was left gave no signal for the Lint, Security, FFmpeg, Go and Rust lanes, named no toolchain or platform version, showed no OpenSSF Scorecard, and carried a github/v/release badge that renders empty — the repository has tags but no GitHub releases.

This restores the set, in the house style of the other CordanaLLM repositories: a banner, a centred header, and badges grouped CI → quality → languages → platforms → project.

Every badge was verified before it landed

No badge here is written from memory.

Check Result
Workflow files behind the seven CI badges all seven exist on master
All 25 badge image URLs HTTP 200
Every local link in the file resolves (four backend links corrected to their real overview.md / index.md paths)
OpenSSF Scorecard live, 8.6, dated today

Versions come from the tree, not from recollection: Go 1.27 (go.mod), Rust edition 2024 (bindings/rust/vmafx-sys/Cargo.toml), Python 3.14 (pyproject.toml), C23 and C++23 (the meson standards), CUDA 13.3.1 / ROCm 10.0.0 / FFmpeg n9.0.1 (build-config.env), SVE2 confirmed present (15 files). The Release badge becomes a tag badge, so it shows v3.0.0 instead of nothing.

Sections

Why VMAFx and Backends at a glance come back as tables — what the fork adds over upstream, and which backend each flag selects — and the quickstart gains the raw-YUV and GPU-backend invocations. The licence and governance sections are carried over verbatim.

docs/assets/vmafx-readme-banner.svg is new: hand-written SVG (no binary asset), the two converging traces being the reference-versus-distorted comparison the tool exists to measure.

Verification

Check Result
mkdocs build --strict exit 0
markdownlint passes
praetorctl audit passes, baseline 1411 → 1411

The five markdownlint rules a centred header needs (MD013 MD023 MD033 MD036 MD041) are disabled at the top of the file with a comment saying why — the same directive style docs/adr/0000-template.md already uses. Prose still wraps at 80.

Type

  • docs — documentation

Checklist

  • Commits follow Conventional Commits.
  • Pre-push hooks pass.
  • I did not modify any assertAlmostEqual(...) score in the Netflix golden Python tests.
  • docs/state.md: no state delta: documentation only; no bug in the tracked set opens or closes.

Deep-dive deliverables (ADR-0108)

  • Research digest — no digest needed: trivial. The verification is the table above.
  • Decision matrix — no alternatives: only-one-way fix. Badges either point at something that resolves or they do not.
  • AGENTS.md invariant note — no rebase-sensitive invariants: the README and its SVG asset.
  • Reproducer / smoke-test command — below.
  • CHANGELOG fragment — changelog.d/added/readme-badges-and-banner.md.
  • Rebase note — no rebase impact: fork-local files an upstream sync never touches.

Reproducer

# every badge URL resolves
grep -oE '!\[[^]]*\]\((https://[^)]+)\)' README.md | sed -E 's/.*\((https[^)]+)\)/\1/' |
  sort -u | while read -r u; do printf '%s %s\n' "$(curl -sS -o /dev/null -w '%{http_code}' -L "$u")" "$u"; done

# every local link resolves
grep -oE '\]\(([a-zA-Z0-9][^):]*\.(md|txt|toml|mod|env|svg))\)' README.md |
  sed -E 's/^\]\(//; s/\)$//' | sort -u | while read -r f; do [ -e "$f" ] || echo "MISSING $f"; done

mkdocs build --strict

@github-actions github-actions Bot added the type:docs Documentation updates label Sep 20, 2026
… two lost sections

The rc1 documentation pass (2026-09-08) cut the README from 22 badges to
four and dropped two sections with it. What was left gave no signal for
the Lint, Security, FFmpeg, Go and Rust lanes, no toolchain or platform
versions, no Scorecard, and a Release badge that renders empty because
the repository has tags but no GitHub releases.

The badge set is back and grouped: CI lanes, quality and governance,
languages, platforms, then project. Every one was verified before it
landed -- all seven workflow files exist, all 25 badge URLs return 200,
and every local link resolves. Facts come from the tree rather than
memory: Go 1.27 from go.mod, Rust edition 2024 from the bindings crate,
Python 3.14 from pyproject.toml, C23 and C++23 from the meson standards,
CUDA 13.3.1 and ROCm 10.0.0 and FFmpeg n9.0.1 from build-config.env, and
the OpenSSF Scorecard reports 8.6. The Release badge becomes a tag badge,
which shows v3.0.0 instead of nothing.

'Why VMAFx' and 'Backends at a glance' return as tables, and a banner
SVG matches the house style of the other CordanaLLM repositories. The
licence and governance sections are carried over verbatim.
Two commits since the last bump: the persona caveman lint (#369) and the
Train B batch repairs (#350). Recording the baseline on master with the
pinned engine and with fae37a3 gives byte-identical infraction sets
(1411 entries, same fingerprints), and the full audit passes under the
new one, so the bump changes no finding.

The comment now states the rule rather than one instance of it: a bump
lands only after the old and new engines are shown to agree.
@lusoris
lusoris force-pushed the docs/readme-badges-and-polish branch from 18ed062 to 4647ce2 Compare September 20, 2026 10:48
@lusoris
lusoris enabled auto-merge (squash) September 20, 2026 13:28
@lusoris
lusoris requested a lite review from Copilot September 20, 2026 13:28

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The changelog is out of sync, and README claims and examples require correction.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 High severity · 2 Medium severity

Open (3)
What changed in this PR

Restores the README’s badges, banner, and documentation sections while updating changelog metadata and the standards-gate pin.

Changes:

  • Restores badges, banner, backend overview, and quickstart examples.
  • Adds the changelog fragment and rendered entry.
  • Updates the Praetor pin and audit baseline timestamp.

Review findings:

  • CHANGELOG.md — critical (2 votes): regenerate the Unreleased block from the fragment.
  • README.md — moderate (3 votes): qualify backend coverage.
  • README.md — moderate (2 votes): correct the bit-exactness claim.
  • README.md — nit (1 vote): correct the upstream/fork comparison.
  • README.md — nit (1 vote): use an explicit GPU extractor or model example.
File Description
README.md Restored badges, banner, documentation sections, and examples
docs/​assets/​vmafx-readme-banner.svg Added README banner artwork
CHANGELOG.md Added rendered changelog entry
changelog.d/​added/​readme-badges-and-banner.md Added changelog fragment
.standards-baseline.json Refreshed audit timestamp
.github/​workflows/​standards-gate.yml Updated Praetor engine pin

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread CHANGELOG.md
Comment thread README.md Outdated
Comment thread README.md Outdated
@lusoris
lusoris merged commit 371ff58 into master Sep 20, 2026
74 of 82 checks passed
@lusoris
lusoris deleted the docs/readme-badges-and-polish branch September 20, 2026 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:docs Documentation updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants