docs: maintainer pass — consume feedback backlog, reconcile GPU dead-ends + stale numbers#156
Merged
Merged
Conversation
…ends + stale numbers Consume the .claude/feedback backlog into CLAUDE.md: - Reconcile GPU dead-ends with pz2-G32 stage 2 data: the old "GPU entropy is slow (0.54x decode)" was a serial-topology artifact — a wire-codesigned Metal kernel hits 64.75 GB/s on literals. GPU block decode still dies, but on the LZ splice / shared-memory ceiling (0.21x at real occupancy, 1.02x parity at 16x saturation), not entropy. Split the encode vs decode framing. - Amend the "no per-core decode win; gap is structural" entry: the wall was format-structural, not hardware — pz2 (sequence wire + 8-lane Huffman) decodes ~3.5x faster per core at parity. Replace the format, don't tune lzf. - Update pz bw decode 1120 -> 1367 MB/s (K=8 multi-cursor iBWT shipped, 2.66x all-cores) and document the multi-cursor rider + GPU-iBWT rejection. - Num row: now auto-routes via -a (#138); add the sample-trial gotcha. - New dead-end: vertical bit-packing can't replace per-plane FSE in Num. - Key conventions: vec![0;n]/alloc_zeroed vs with_capacity+resize on decode bufs. - Test caveat: lz77 batched-pipeline perf assertion flakes under machine load. Delete the 14 consumed notes (6 March notes already covered by current dead-ends; ci-clippy fix already landed; numeric-decorr spike superseded by the shipped Num pipeline). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.
Maintainer pass consuming the
.claude/feedback/backlog into CLAUDE.md. Docs-only.Promoted (CLAUDE.md edits)
-anow auto-routes vianumeric_gain(feat(analysis): auto-route fixed-stride numeric data to the Num pipeline #138); added the "don't trial-encode a 64KB sample" gotcha.vec![0;n]/alloc_zeroedvswith_capacity+resizeon fresh decode buffers.test_lz77_batched_pipeline_not_slower_than_serialflakes under multi-agent load (verified: passes in isolation).Discarded / already-covered (notes deleted)
--all-targetsfix already landed in its own PR.All claims verified against
docs/design-docs/pz2-g32-stage2-findings.md,ibwt-cursor-findings.md,num-bitpack-stage0-findings.md, andclean-slate-codec.md.CI note: the one local test failure is the documented flaky wall-clock GPU perf assertion (758/758 real tests pass; flake passes in isolation at 0.21s). Change is docs-only.
🤖 Generated with Claude Code