Repository navigation
docs(readme): restore the badge set, add a banner, and bring back the two lost sections - #1508
Merged
Merged
Conversation
… 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
force-pushed
the
docs/readme-badges-and-polish
branch
from
September 20, 2026 10:48
18ed062 to
4647ce2
Compare
lusoris
enabled auto-merge (squash)
September 20, 2026 13:28
There was a problem hiding this comment.
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
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.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


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 agithub/v/releasebadge 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.
masteroverview.md/index.mdpaths)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). TheReleasebadge becomes a tag badge, so it showsv3.0.0instead of nothing.Sections
Why VMAFxandBackends at a glancecome 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.svgis new: hand-written SVG (no binary asset), the two converging traces being the reference-versus-distorted comparison the tool exists to measure.Verification
mkdocs build --strictpraetorctl auditThe 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 styledocs/adr/0000-template.mdalready uses. Prose still wraps at 80.Type
docs— documentationChecklist
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)
AGENTS.mdinvariant note — no rebase-sensitive invariants: the README and its SVG asset.changelog.d/added/readme-badges-and-banner.md.Reproducer