Repository navigation
D-RU-6 lab: block-sparse tile recursion (arm C) vs walk_mask and the dense tile path - #1648
Conversation
…rojection No behaviour change; the 24 walk_tile tests are unchanged. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01G22yT6htkcdyXsihxxXdrg
…rse) Same binding and projection as walk_tile; the product skips empty tile triples, unoccupied m x m delta blocks and absent rows, with no broadcast, spread or per-coordinate Ternlog. Every index is a loop coordinate. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01G22yT6htkcdyXsihxxXdrg
…_tile Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01G22yT6htkcdyXsihxxXdrg
…aive delta goes red) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01G22yT6htkcdyXsihxxXdrg
Count-only resident memory arms now skip decoding. Block-sparse C beats the dense tile path everywhere and walk_mask end to end only on deep, low-fan-out recursion. D-RU-6 stays Queued/Lab. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01G22yT6htkcdyXsihxxXdrg
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
Warning Billing warning: we have not been able to collect payment for this subscription for more than 72 hours. Please update the payment method or pay any pending invoices in Billing to avoid service interruption. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013z2e1YXkm6o6HNrDn5BPWA
Lab only. D-RU-6 stays Queued/Lab; nothing is admitted.
Follows #1647, which is merged. The base is
main. Nothing was left behind on #1647's branch:git log origin/main..origin/ccr-0455e606-wmtsoris empty, and this branch merges cleanly into main.What this adds
crates/crepe-plan-probe/examples/walk_sparse.rs, arm C. It compileswalk_tile.rs(arm B, #1647), which compileswalk_mask.rs(arm A, #1642). All three arms run on the same graphs and are checked against the same oracles.C keeps B's binding, witness and projection. Its product differs from B's in five ways:
m × mblocks of the delta (m ∈ {16, 32, 64, 256}).Ternlogper coordinate. mask-risc runs the per-tile update instead (AndNot,Or), andadmitrefusesGatherin both programs.walk_tile.rschanges only in visibility (pub(crate)) and a rustfmt pass. Its 24 tests are unchanged.Results (release, medians of 5, ms)
The full tables are in
.claude/board/entries/2026-10-21-d-ru-6-sparse-compute.md: phases, work counts, microtile sweep, memory and reads by origin.Tests
38 tests, including B's 24 and A's 8. They cover:
CI gains
cargo test --example walk_sparse.Disable runs
All were committed first, then reverted.
On the naive-delta disable: the result stays correct, so only the bound test catches it. The bound test's first version used a chain and stayed green, because a chain has no alternative derivations.
What this does not show
No merge on green alone. Coordination: EXO-50.
🤖 Generated with Claude Code
https://claude.ai/code/session_01G22yT6htkcdyXsihxxXdrg