-
Notifications
You must be signed in to change notification settings - Fork 0
facet: revert the axis LCP to the PEEK chain; name the three prefix-fold carriers #1245
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
a714234
b08db9b
5984e85
c3de1d2
6e79c78
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,17 @@ | ||
| ## three-carrier prefix folds (D-ids minted 2026-09-17, plan `.claude/plans/three-carrier-blast-radius-v1.md`) | ||
|
|
||
| Arose from the −32 offset correction sweep (#1244) and the operator's challenge to it. | ||
| Doctrine: `.claude/knowledge/three-prefix-fold-carriers.md`. Board: | ||
| `E-THREE-CARRIERS-THREE-FOLDS-1`. | ||
|
|
||
| | D-id | scope | status | gate / falsifier | | ||
| |---|---|---|---| | ||
| | D-TCF-1 | probe the facet per-axis LCP four ways and settle which fold the byte-addressed carrier wants | **Shipped.** `examples/facet_axis_lcp_probe.rs`, 64K pairs, min-of-7, oracle-first. Chain PEEK **1.72 ns** vs masked `u128` **3.64 ns** on random; PEEK wins at every workload; the two pack-to-register arms are slowest | all four arms must agree with the shipped API before timing; the depth knob must bind (A: 1.66 → 3.50 ns over depth 0..5, +111%) — a flat slope would mean the early exit is not the mechanism | | ||
| | D-TCF-2 | revert `shared_axis` to the chain fold; keep the differential test falsifiable | **Shipped.** Masked form retained as `masked_axis_oracle` under `#[cfg(test)]` — direction reversed so the test compares shipped-vs-oracle, not shipped-vs-itself | post-revert re-run reproduces the ordering (A 1.76 / C 4.48); 1354 contract tests green; the prior disable-run (swap hi/lo masks ⇒ fail at `hi flip at tier 0`) still holds on the oracle | | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win 🔎 Supported by static analysis🏁 Script executed: printf '%s\n' '--- status board ---'
cat -n .claude/board/STATUS_BOARD.md | sed -n '1,40p'
printf '%s\n' '--- references to counts ---'
rg -n -C 3 '1,425|1425|1,354|1354|contract tests|contract-test' . --glob '!node_modules' --glob '!dist' --glob '!build'Repository: AdaWorldAPI/lance-graph Length of output: 50380 🤖 get_repo_knowledge executed:
Length of output: 10068 🏁 Script executed: printf '%s\n' '--- status entry ---'
rg -n -C 2 'D-TCF-2|contract tests|contract-test' .claude/board --glob '*.md'
printf '%s\n' '--- exact count references in documentation ---'
rg -n -C 2 --glob '*.md' --glob '*.yml' --glob '*.yaml' --glob '*.toml' --glob '*.json' '1,425|1425|1,354|1354|contract tests|contract-test' . || true
printf '%s\n' '--- repository state ---'
git status --short
git diff --statRepository: AdaWorldAPI/lance-graph Length of output: 50379 Align the contract-test count. The PR objective reports 1,425 passing contract tests, while D-TCF-2 reports 1,354 for the post-revert re-run. Label each count with its scope or commit, or update the status entry with the correct count. 🤖 Prompt for AI Agents |
||
| | D-TCF-3 | name the three carriers and the law that separates them | **Shipped.** `.claude/knowledge/three-prefix-fold-carriers.md` — bit-planes / nibble path / facet cascade; *masking wins when the slice is granular, PEEK wins when the slice is addressed* | falsified by a fourth carrier, or by one carrier consuming another's fold. Two greps must stay empty: `FacetCascade` in `mailbox_soa.rs`; `trailing_zeros\|leading_zeros` in `hhtl.rs` | | ||
| | D-TCF-4 | carrier-2 (`NiblePath::common_prefix_depth`) masked rewrite | **Queued — CONJECTURE, deliberately unbuilt.** `ISS-NIBLEPATH-FOLD-IS-CARRIER-2-UNMASKED`. This is the carrier the 2026-09-16 instinct was right about | must be probed on ITS OWN carrier first (G-PROBE). Landing it on entry (16)'s number would be the original defect recurring | | ||
| | D-TCF-5 | blast-radius census of all three carriers + their seams | **Queued.** Plan above, passes 1–4, read-only | G-EXH (walk, not a chosen file list) · G-FOLD (every fold site carries a carrier) · G-SEAM (owner-in / owner-out / cost) · G-ZERO (a "no callers" claim verified by opening, never by an empty grep) | | ||
|
|
||
| ## elk/ro factfinder — the meet, the horizon and their generality (D-ids minted 2026-09-15, probe `.claude/probes/elk-generality-v1/`) | ||
|
|
||
| Arose from the EWA 12-hop fanout question ("would MQ offer a cheap gating"). No plan | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
🔎 Supported by static analysis
🏁 Script executed:
Repository: AdaWorldAPI/lance-graph
Length of output: 2240
Keep the carrier-2 mask classification provisional.
The carrier table already labels masking as
✗ opportunity, and the issue body states that the rewrite isCONJECTURE until probed. Replace “genuinely wants a mask” in the issue title with provisional wording such as “may benefit from a mask.” No change is needed to the carrier-table or conjecture wording.🤖 Prompt for AI Agents