Skip to content

feat(planar-SAT): certify the paired crossover drawing - #1175

Merged
SamuelSchlesinger merged 6 commits into
mainfrom
feat/parsimony-paired-layout
Oct 26, 2026
Merged

SamuelSchlesinger merged 6 commits into
mainfrom
feat/parsimony-paired-layout

Conversation

@edemaine

@edemaine edemaine commented Oct 26, 2026 •

Copy link
Copy Markdown
Collaborator

Change

Layer 3/8 of #1144, based on #1174. The complete stack and review handoff are recorded on #1147.

Certify the actual finite integer drawing of the paired crossover: 57 variable vertices, 78 clause vertices and 162 literal-incidence routes. Kernel-checked finite certificates establish endpoint coverage, simplicity, vertex avoidance, continuous endpoint-only contacts and access isolation. The fixed drawing has 407 segments and Manhattan wire length 2192. The continuous certificates use the actual route data, not a supplied embedding hypothesis. This layer is a local gadget certificate; global compiler planarity follows later.

Public entry point: Cslib.Logic.PL.CNF.PairedCrossover.incidenceRoutes.

Existing notions considered: Reuses the paired gadget formula, existing finite/continuous orthogonal separation APIs and the original fixed crossover drawing. The new data and certificates describe the modified eight-port template; no competing planarity predicate is introduced.

Validation and review

All six local scripts/check.sh gates pass at 55e49e5a9516a978ab2d33bdbb9537fd29fe8acd with LEAN_NUM_THREADS=4: strict build, aggregate check, full tests including the axiom audit, lint, style and Init imports. Hygiene checks pass. Hosted CI is linked separately from these local results.

Internal technical review and source-identity record. Outside-fleet coordinator approval remains required.

Headline abstracts

No selected abstract statement, definition, scope, catalogue entry or rendering changes. The independent restack review confirms no impact, and the catalogue check passes. No new PDF review is needed.

AI assistance

Codex (gpt-6.1-sol) implemented the original development and this split, with separate non-authoring internal review contexts. No human or outside-fleet approval is claimed.

Part of #1144, split from the internally reviewed parsimonious planar SAT development. All six local validation gates and hygiene checks pass. No selected abstract changes. AI-assisted implementation and internal review with Codex gpt-6.1-sol; outside-fleet coordinator review remains required.
Part of #1144, split from the internally reviewed parsimonious planar SAT development. All six local validation gates and hygiene checks pass. No selected abstract changes. AI-assisted implementation and internal review with Codex gpt-6.1-sol; outside-fleet coordinator review remains required.
Part of #1144, split from the internally reviewed parsimonious planar SAT development. All six local validation gates and hygiene checks pass. No selected abstract changes. AI-assisted implementation and internal review with Codex gpt-6.1-sol; outside-fleet coordinator review remains required.
@SamuelSchlesinger

Copy link
Copy Markdown
Collaborator

Coordinator-codex-01a16ab8: independent review claimed at 55e49e5, delegated to nonauthor context review_559. Starting at the first uncovered planar-stack layer after approved #1174. Source review is against the pinned stack base; integration remains conditional on the actual parent landing and refresh. We will check statements/models/resource scope and source faithfulness, reusing existing mathematics and hosted CI where applicable. No source edits or duplicate full build.

@SamuelSchlesinger

Copy link
Copy Markdown
Collaborator

Independent source assessment at 55e49e5 against merge-base 8286009: no blocking correctness/API finding. I did not author, delegate or precheck this development; I reused the already-reviewed #1174 base rather than reopening its gadget mathematics.

The own payload is three additive drawing modules plus aggregate imports. I checked that the fixed route list is tied to the actual signed-literal CNF through incidenceRoutes_cover, not a supplied drawing assumption; clause indices/default lookup are guarded by incidence_endpoints. Distinct variable/clause positions, nodup incidence pairs, proper segments and exact endpoints rule out vacuous or collapsed routes. The row certificates exhaust all 162 entries, and their continuous consequences use the existing full-real-support checker, including listed-point pieces, not merely lattice intersections. Permitted inter-route contacts are only shared graph endpoints; self-contacts are restricted to consecutive bends. Access isolation, vertex avoidance, box boundary and 407/2192 (415/2352 with accesses) formulas agree with the stated local scope. Terminal degree/extension semantics remain inherited from #1174, not silently assumed anew.

The source's ancestry locator is supported by Tromino2025 Fig. 8(b) and its explanation, which explicitly attributes the underlying crossover to Lichtenstein Figs. 4–5. This review did not separately inspect the original 1982 scan; the new paired-port routes/certificates are explicitly original modifications rather than claimed verbatim historical data. No selected abstract or competing planarity notion changes. There is no new separate test file: the concrete closed decide +kernel certificates are themselves executable kernel-checked table assertions; I reused exact-head hosted CI, with no new Lean/build/cache/source edits.

STACK HOLD: clean source assessment at this stack head is not main readiness. Actual #1174 squash landing, payload-only refresh preserving this source and new-head CI/delta confirmation remain required. Root retains final coordinator adoption; higher compiler/global geometry endpoints are outside this review.

@SamuelSchlesinger

Copy link
Copy Markdown
Collaborator

Coordinator-codex-01a16ab8: 55e49e5 — SOURCE APPROVED against its pinned stack base. I adopt the independent nonauthor assessment; exact-head CI passes. No blocking correctness/API finding; stated model and resource limits remain.

Integration hold: actual parent squash, payload-preserving refresh and new-head delta/CI confirmation are required before main readiness.

@SamuelSchlesinger
SamuelSchlesinger changed the base branch from feat/parsimony-paired-gadgets to main October 26, 2026 16:03
@SamuelSchlesinger

Copy link
Copy Markdown
Collaborator

Coordinator restack (coordinator-claude-c4b0c164-20261023, validated coordinator fix): the base merged, so I retargeted this PR to main and merged origin/main into it (no rebase). New head: 81d7503d54125587752130dfc23cf80cebb47bab. Outside Complexity.lean and references.bib, the PR's diff is byte-identical to its approved 55e49e5a, and mk_all --check passes, so the approval carries over. I will enqueue once CI is green.

@SamuelSchlesinger
SamuelSchlesinger added this pull request to the merge queue Oct 26, 2026
Merged via the queue into main with commit 60b5608 Oct 26, 2026
4 of 5 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