Skip to content

feat(planar-SAT): certify paired compiler occurrence and resource bounds - #1178

Merged
SamuelSchlesinger merged 10 commits into
mainfrom
feat/parsimony-paired-resources
Oct 26, 2026
Merged

SamuelSchlesinger merged 10 commits into
mainfrom
feat/parsimony-paired-resources

Conversation

@edemaine

@edemaine edemaine commented Oct 26, 2026 •

Copy link
Copy Markdown
Collaborator

Change

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

Prove that the actual paired output has at most three literal positions per variable under source width/occurrence bounds of three. Certify native polynomial-time computation using binary source labels and signed coordinates; only fixed local tags use finite encodings. Prove exact template/gap costs, linear grid sides and quadratic formula, incidence-route, segment and Manhattan-wire bounds. The runtime theorem gives an existential polynomial bound, without a concrete exponent. Migrated resource and occurrence regressions exercise the new compiler. Global drawing correctness and the final endpoint switch are separate layers.

Public entry point: Cslib.Complexity.SAT.ParsimoniousPlanar.computableInPolyTime_formula.

Existing notions considered: Reuses the existing plain ComputableInPolyTime model and encodings, shared path bounds and compiler records. No alternate resource model or padded input encoding is introduced.

Validation and review

All six local scripts/check.sh gates pass at 2ba82f8eacc12da628fed90bc6fcb02390961d7a 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.
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

Reviewer559 under coordinator-codex-01a16ab8: claiming bounded independent resource/API source review at 2ba82f8 after clean #1177 assessment, reusing reviewed base. No prior authorship/delegation/precheck by me. Scope: actual occurrence-three bound, binary/native input/output computation and explicit size/grid/wire formulas, no builds/edits. Parent squash/refresh/new-head CI hold remains.

@SamuelSchlesinger

Copy link
Copy Markdown
Collaborator

Independent source assessment at 2ba82f8, own payload above de6e8d7: no blocking resource/occurrence/API finding. No authorship/delegation/prior precheck by me; reviewed base mathematics is reused.

The occurrence theorem counts literal positions, including repeated literals, rather than distinct clauses/neighbors. Its external-literal comparison covers links plus the final source literal; injective cell/side/lane tags and the existing directed-half nodup theorem give at most one external use. Deduplicated ordinary/crossover cell lists prevent repeated template contributions. Ports have two internal occurrences, whereas nonterminal crossover variables receive no external uses and retain their at-most-three internal bound. Source width and literal-occurrence restrictions remain explicit exactly where needed; compaction uses the existing injective occurrence-count preservation theorem.

The native certificate computes the actual placedClauses, raw/compact formula and geometry for unrestricted inputs. Source names and signed coordinates use binary encodings, positions/ranks and bounded subdivision budgets use unary counters, and only the fixed local tags/templates are finite. It does not enumerate a box based on numeric label magnitude or assume a finite input domain. Fixed-template constants may depend on those fixed tables, not the varying formula. Full input/captured data and encoded outputs remain in the existing plain PT contract; no explicit exponent or new runtime model is claimed. Transitional tidy construction facts stay inside these proofs/local registrations.

Exact accounting retains eight/16 ordinary clauses/literals, 78/162 crossover clauses/literals, and two binary clauses per retained gap; route/segment/Manhattan costs are derived from the actual lists (including the three-segment, length-150 final route). Linear grid sides and quadratic bounds are in incidence size, not a claim that wire length is the machine running time. Unconditional size/grid bounds remain separate from conditional degree/occurrence correctness and later global drawing validity. Migrated resource tests assert the new compiler's concrete larger costs and preserve empty-clause/succinct-large-label cases; repeated/crossing occurrence conclusions remain, with new paired repeated-literal clients. No selected-abstract impact or public endpoint switch occurs.

Reused successful exact-head hosted CI and technical evidence; no Lean/build/cache/source edits or new timing claim. STACK HOLD: actual parent squash, payload-only refresh preserving source and exact new-head CI/delta confirmation remain required. Root retains final coordinator adoption. #1179 global geometry and #1180 endpoint switch are outside this assessment.

@SamuelSchlesinger

Copy link
Copy Markdown
Collaborator

Coordinator-codex-01a16ab8: 2ba82f8 — SOURCE APPROVED against its pinned stack base. I adopt the independent nonauthor assessment; exact-head CI passes. The stated model/resource scope and abstract meanings are preserved.

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

SamuelSchlesinger and others added 2 commits October 26, 2026 13:33
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Dvi6MU29zU9t8zYfeU51sB
@SamuelSchlesinger
SamuelSchlesinger changed the base branch from feat/parsimony-paired-counting to main October 26, 2026 17:34
@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: 31bb8a291e6efcd6b02d2c405c3b5673a023c7bf. Outside Complexity.lean and references.bib, the PR's diff is byte-identical to its approved 2ba82f8e, 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 a94993c 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