Skip to content

pillar11 W0 (ndarray half): battery claim reframed; sigker_hl misnomer corrected - #289

Merged
AdaWorldAPI merged 1 commit into
masterfrom
claude/pillar11-w0-ndarray
Aug 31, 2026
Merged

pillar11 W0 (ndarray half): battery claim reframed; sigker_hl misnomer corrected#289
AdaWorldAPI merged 1 commit into
masterfrom
claude/pillar11-w0-ndarray

Conversation

@AdaWorldAPI

Copy link
Copy Markdown
Owner

What

The ndarray half of W0 (green-lit 2026-08-31; ruling Q3: docs only — no registry slot-id change), resolving census findings F-1 and F-2 from lance-graph's pillar11-signature-certification-unification-v1:

  • F-1 (name collision, disambiguated in place): the module headline claimed "Hambly–Lyons signature transform"; what prove_pillar_11 actually certifies is truncated signature-kernel stability (PSD + concentration of the depth-3 Gram over 1000 Brownian paths). Reframed, with the cross-repo note spelled out: lance-graph jc's Pillar 11 certifies the HL uniqueness theorem; this one certifies kernel stability — two repos, one slot number, disjoint claims, and a session reading "Pillar 11 is green" must know which. The B7/11 slot id is stored history, deliberately unchanged.
  • F-2 (misnomer): sigker_hl labelled a plain truncated sig-kernel a "Hambly–Lyons kernel"; HL 2010 is a uniqueness theorem, not a kernel construction. The _hl symbol survives for API stability; the docs stop carrying the imported authority.

Companion PR (lance-graph half, F-3 + the Q1–Q3 record): lance-graph #1111.

Verification

Docs-only diff; workspace cargo check green.

🤖 Generated with Claude Code

https://claude.ai/code/session_01KCGhDYoQBXs3poaR7sFuqp


Generated by Claude Code

…hl misnomer corrected

W0 of lance-graph's pillar11-signature-certification-unification-v1,
green-lit 2026-08-31, ruling Q3: docs only.

- The module headline claimed the Hambly-Lyons signature transform; what
  the battery certifies is truncated signature-kernel STABILITY (PSD +
  concentration over Brownian paths). Reframed, with the cross-repo
  disambiguation spelled out: lance-graph jc's Pillar 11 certifies the
  HL uniqueness theorem, this one certifies kernel stability - two
  repos, one slot number, disjoint claims. The B7/11 slot id is stored
  history and unchanged.

- sigker_hl labelled a plain truncated sig-kernel a Hambly-Lyons
  kernel; HL 2010 is a uniqueness theorem, not a kernel construction.
  The symbol survives for API stability; the docs stop carrying the
  imported authority.

Docs-only diff; workspace check green.

Co-Authored-By: Claude <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KCGhDYoQBXs3poaR7sFuqp
@coderabbitai

coderabbitai Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Next included review available in 55 minutes.

View limit details

Limit details: You’ve used all 2 included reviews currently available. Your 58 included PR review attempts over the past 7 days set your current allowance at 2 reviews per hour.

Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 53d40a3a-34fa-40d3-8294-ffaa7528be9b

📥 Commits

Reviewing files that changed from the base of the PR and between 61a2695 and fbbfcd8.

📒 Files selected for processing (1)
  • src/hpc/pillar/signature.rs

Comment @coderabbitai help to get the list of available commands.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: fbbfcd8f10

ℹ️ 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".

Comment on lines +5 to +6
//! This battery certifies **kernel stability**: PSD-ness + concentration of
//! the depth-3 truncated signature-kernel Gram over Brownian paths. It does

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Do not claim that this probe certifies PSD

For the documented Brownian-path run, prove_pillar_11 only checks positive self-kernels and pairwise Cauchy–Schwarz inequalities on 50 paths; those are necessary but not sufficient conditions for a 50×50 matrix to be PSD, and the implementation never performs the Sylvester test claimed later in the module docs. Consequently, a passing report does not empirically certify the PSD property newly advertised here; either perform a complete PSD check or narrow this claim to the conditions actually tested.

Useful? React with 👍 / 👎.

Comment on lines +38 to +40
//! signatures — a plain truncated sig-kernel. The "Hambly–Lyons" label this
//! file attached to it is a MISNOMER kept only in the function name for API
//! stability (`sigker_hl`): HL 2010 is a *uniqueness theorem*, not a kernel

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Remove the remaining Hambly–Lyons labels

The statement that the misnomer is retained only in sigker_hl is contradicted by the generated documentation in this same module: it still describes a “Hambly–Lyons Gram matrix,” documents PILLAR_11_SEED as belonging to the Hambly–Lyons transform, and calls prove_pillar_11 a Hambly–Lyons sigker probe. Users entering through those public items therefore still receive the attribution this change is intended to correct; update those remaining labels or avoid claiming that only the symbol retains it.

Useful? React with 👍 / 👎.

@AdaWorldAPI
AdaWorldAPI merged commit 9b3e147 into master Aug 31, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants