Skip to content

docs(state): move four merged-fix rows out of Open bugs - #1292

Merged
lusoris merged 1 commit into
masterfrom
docs/state-sweep-four-closed-rows
Sep 5, 2026
Merged

lusoris merged 1 commit into
masterfrom
docs/state-sweep-four-closed-rows

Conversation

@lusoris

@lusoris lusoris commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Summary

Bookkeeping sweep of docs/state.md: four rows sat under ## Open bugs although their fixes are merged on master. Three rows move to ## Recently closed with a CLOSED <date> status column citing the merged PR, commit and code evidence: T-HIP-PSNR-CHROMA-MCP-PARITY-2026-06-20 (PR #1022, 6c0d70441, merged 2026-06-20 — enable_chroma option in integer_psnr_hip.c:102-109 / float_psnr_hip.c:105-112, MCP backend enum parity in cmd/vmafx-mcp/impl.go:223-225 and server.py:483), T-CI-TOX-PY311-SCIPY-118-2026-06-20 (PR #1023, 5dbc9ebfa, merged 2026-06-27 — python/tox.ini:2 envlist = py314, coverage, CI Python 3.14.7, master Build run 33916590335 macOS tox step success), and T-CI-DOCKER-SMOKE-NO-OUTPUT-2026-06-13 (PR #903, b9eb49e79 — docker-image.yml:57-118 writes --output /out/result.json and asserts via Python; master Docker Image Build runs green 7/8, latest 33913809345). The duplicate Open copy of T-NEON-FMA-FLOAT-ADM-DWT2-2026-06-06 is deleted (the CLOSED 2026-08-30 row for PR #1161 195f88a22 already exists in Recently closed) and the _Updated: header is refreshed. Every citation above was re-verified by the shipping reviewer against origin/master (git show -s, gh pr view, sed -n on the cited lines, gh run view). The HIP row notes that no standalone HIP build CI workflow exists, so the old "CI HIP leg builds" claim stays unverified; the code evidence is conclusive. Rows for T-HIP-MOTION-V2 (PR #1279) and T-UPSTREAM-1564 (in-flight adm bundle) were deliberately left untouched. No code changes.

Type

  • docs — docs/state.md bookkeeping: move four merged-fix rows out of Open bugs

Checklist

  • Commits follow Conventional Commits.
  • make format && make lint is green locally (pre-commit on every touched file).
  • Unit tests: n/a: docs-only change, no code touched.
  • Docs in the same PR: docs/state.md, docs/rebase-notes.md, CHANGELOG.md (regenerated from changelog.d/).
  • SIMD/GPU: n/a; twins: n/a; new C sources: n/a; breaking change: n/a; ADR: n/a (bookkeeping, no decision).

Bug-status hygiene (ADR-0165)

  • docs/state.md — T-HIP-PSNR-CHROMA-MCP-PARITY-2026-06-20, T-CI-TOX-PY311-SCIPY-118-2026-06-20 and T-CI-DOCKER-SMOKE-NO-OUTPUT-2026-06-13 moved from Open bugs to Recently closed; duplicate Open copy of T-NEON-FMA-FLOAT-ADM-DWT2-2026-06-06 deleted; _Updated: header refreshed.

Netflix golden-data gate (ADR-0024)

  • I did not modify any assertAlmostEqual(...) score in the Netflix golden Python tests.

Deep-dive deliverables (ADR-0108)

  • Research digest — no digest needed: trivial docs/state.md bookkeeping, no code or design change.
  • Decision matrix — no alternatives: only-one-way fix. Rows whose fixes are merged belong in Recently closed per the state.md update protocol.
  • AGENTS.md invariant note — no rebase-sensitive invariants: docs-only bookkeeping.
  • Reproducer / smoke-test command — below.
  • CHANGELOG fragment — changelog.d/fixed/state-sweep-2026-09-04.md
  • Rebase note — docs/rebase-notes.md entry (no rebase impact: docs-only).

Reproducer

# The three moved ids appear exactly once as a table row, all after the Recently closed heading
for id in T-HIP-PSNR-CHROMA-MCP-PARITY-2026-06-20 T-CI-TOX-PY311-SCIPY-118-2026-06-20 T-CI-DOCKER-SMOKE-NO-OUTPUT-2026-06-13 T-NEON-FMA-FLOAT-ADM-DWT2-2026-06-06; do
  echo "$id: rows=$(grep -c "^| \*\*$id\*\*" docs/state.md) closed_heading=$(grep -n '^## Recently closed' docs/state.md | cut -d: -f1) row_line=$(grep -n "^| \*\*$id\*\*" docs/state.md | cut -d: -f1)"
done
# expected: rows=1 for each id, row_line > closed_heading for each id
bash scripts/release/concat-changelog-fragments.sh --check; echo "concat rc=$?"      # concat rc=0
PR_TITLE="fix: state sweep" bash scripts/ci/state-md-touch-check.sh | tail -1        # state-md-touch-check: PASS — docs/state.md is in the diff (no placeholder refs in inserted lines).
pre-commit run --files ./docs/state.md ./changelog.d/fixed/state-sweep-2026-09-04.md ./CHANGELOG.md ./docs/rebase-notes.md   # all hooks Passed

🤖 Generated with Claude Code

@lusoris lusoris added this to the 1.0.0 — First release milestone Sep 5, 2026
@lusoris
lusoris force-pushed the docs/state-sweep-four-closed-rows branch from 1dcf55e to e0a462e Compare September 5, 2026 15:22
@lusoris
lusoris marked this pull request as ready for review September 5, 2026 15:22
@lusoris
lusoris merged commit f337f64 into master Sep 5, 2026
107 of 108 checks passed
@lusoris
lusoris deleted the docs/state-sweep-four-closed-rows branch September 5, 2026 15:49
@lusoris lusoris added the type:docs Documentation updates label Sep 7, 2026
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.

1 participant