cognitive-shader-driver: emit CE64 from the SPOFC record (land #1294 on main) - #1295
Conversation
Stage [5] packed every edge from the hit alone: S = row%256, O = (row/4)%256 (a row id in palette-address space), f = c = resonance, the p64 predicate bits raw as the Pearl mask, and topology left at the packed default Direct. Now each edge is packed from its candidate's SPOFC record: - S / O are palette256 indices: the row's stored archetype (the value the cascade queries with) and the partner's (a P64 target, or a row partner's stored archetype). P stays 0: no predicate palette exists. - f / c are the SPOFC truth; confidence counts supporting relations. - The Pearl mask is the inverse of edge_to_layer_mask's causal half. `predicates & 7` put SUPPORTS on bit 2, which reads back as CONTRADICTS. - Topology says how the relation was found: a P64 target is Direct, a content-similarity partner is Unknown. Adds ce64_recount_probe: learn() pools a repeated observation as fresh evidence, and the driver's ALL_FROZEN emission plasticity stops learn() moving S/P/O. Both are characterisations for the cross-cycle work. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019HnekoM1EidTwQLS3oFVFm
The round-trip test covered layers_to_causal_mask, but emission could still pass the raw bits: the CAUSES fixture maps to bit 0 either way. Moved the fixture to a SUPPORTS plane, which has no Pearl bit. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019HnekoM1EidTwQLS3oFVFm
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019HnekoM1EidTwQLS3oFVFm
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019HnekoM1EidTwQLS3oFVFm
…l-gni3cw-ce64-spofc cognitive-shader-driver: emit CE64 from the SPOFC record (follow-up to #1293)
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Essentials Run ID: 📒 Files selected for processing (6)
✨ Finishing Touches📝 Generate docstrings
Comment |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_74995a8b-c04d-4a58-b4e7-1788f728f496) |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fa9a9b09a2
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| spofc.truth.frequency, | ||
| spofc.truth.confidence, | ||
| layers_to_causal_mask(spofc.predicates), |
There was a problem hiding this comment.
Keep evidence scoped to the selected object
When one row has supports to different objects or through different predicate planes, CandidateTable keeps only the highest-resonance object but unions predicates and counts support across every relationship. Packing that aggregate here therefore attributes unrelated predicate bits and inflated confidence to the single chosen S/O pair; for example, a strong CAUSES hit to target A plus a weaker CONTRADICTS hit to target B emits target A as both CAUSES and CONTRADICTS, and edges_to_layered_rows materializes the fabricated contradiction. Aggregate evidence per object, or emit separate edges, before encoding it as a binary CE64 relation.
Useful? React with 👍 / 👎.
…l-gni3cw-revert-1295 Revert #1295
Why this PR exists. #1294 was stacked on #1293's branch, and GitHub merged it into that branch at 19:32:16Z. #1293 had already merged to
main17 seconds earlier, so #1294's four commits (plus its merge commit) are stranded onclaude/llvm-codegen-polyfill-gni3cwand are not onmain:This PR carries exactly those commits to
main. There are no new changes. The content, tests and rationale are the ones reviewed in #1294; its description applies unchanged.What lands
Emitted CausalEdge64 (CE64) is packed from the SPOFC record:
edge_to_layer_mask. The rawpredicates & 7had turned SUPPORTS into CONTRADICTS.Directfor a P64 target,Unknownfor a content-similarity partner.Also included:
tests/ce64_recount_probe.rs: characterises thatlearn()recounts a repeated observation, and that emission plasticity isALL_FROZEN..claude/plans/ce64-spofc-learning-v1.md, with deliverables D-CSL-A…F and theirSTATUS_BOARDrows. The blocked legs name their missing dependencies.Checks
All 10 checks were green on
04315c74when #1294 merged. The only commit since is the merge commitfa9a9b09. CI re-runs here againstmain.🤖 Generated with Claude Code
https://claude.ai/code/session_019HnekoM1EidTwQLS3oFVFm
Generated by Claude Code