Add the shared pull-request template (taphouse sync) - #30
Conversation
Synced from the canonical TapHouse copy: the family's shared review prompts -- what changed and why, a Verification section asking what was actually built and run versus what CI will gate (plus the measured-not-remembered rule for performance claims), and a delete-what-does-not-apply list of the recurring cross-repo concerns: contract changes, submodule pin flow, notebook re-execution, package docs/help and universal binaries, and documented per-repo exceptions. Created only-if-missing and deliberately NOT drift-guarded, following the .claude/settings.json precedent: it is prose for humans rather than a machine-enforced config, so this repo is free to tailor it -- trim the bullets that do not apply here. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JhhQ93r2E1QTnCx46YfX8j
|
CI is red here on macOS AppleClang, and it is not caused by this PR. Flagging it once with the evidence rather than papering over it. Why this PR isn't the cause. The entire diff is one file, What changed is the runner image. The macOS runners now report AppleClang 21.0.0 / Xcode 26.5 and ship CMake 4. That image move is independently corroborated: the same update broke Scope: macOS-only, and specific to this repo's float32 gates. Green in the same run: Linux Clang, Linux GCC, Windows MSVC, Hexagon cross (QEMU), Cortex-M55 cross (QEMU), ASan + UBSan, the instruction-count ratchet, clang-format and drift. And it is not a blanket AppleClang 21 problem across the family — tap/AmbiTap#24's Worth a real look, not a tolerance nudge. 42.34 dBm0(A) against a −55 gate is roughly a 97 dB miss. That is far too large for floating-point drift, so the narrowband guard or suppressor is likely not engaging at all on this codegen rather than merely landing slightly off. Two float32 tests failing together points at the float32 path systematically. Not fixing it in this PR. It is unrelated to the template, and I cannot reproduce it: Linux Clang and GCC both pass, and I have no macOS available in this environment, so any change would be guesswork rather than a diagnosis. It deserves its own issue and a reproduction on the new toolchain — happy to open one with this evidence if that is useful. Generated by Claude Code |
|
Tracked as #31, with a one-flag bisection to try first: macOS is the only platform where the float32 FFT routes through vDSP rather than Ooura ( This PR itself is unaffected — one markdown file, no C++. Generated by Claude Code |
Syncs the new canonical pull-request template from TapHouse (see tap/TapHouse#5, which adds it to the canonical set and to
scripts/sync.sh). None of the 13 family repos had one.Single file, no code or CI changes.
The template asks for what changed and why, a Verification section stating what was actually built and run versus what CI will gate (plus the family's measured-not-remembered rule for performance claims), and a delete-what-does-not-apply list of the recurring cross-repo concerns.
Relevant here: the measured-not-remembered rule in Verification, given this repo's ITU-T compliance battery and the instruction-count ratchet under
bench/icount/— a PR that moves a workload's count should say so and cite the number; and notebook re-execution, noting this repo's notebooks are generated bytools/notebook/build_*.pyrather than hand-edited, which is worth stating in a PR that touches them.It is created only-if-missing and deliberately not drift-guarded, following the
.claude/settings.jsonprecedent rather than the.clang-formatone: it is prose for humans, so this repo is free to tailor it. Trim the bullets that don't apply.Generated by Claude Code