Skip to content

refactor(planar-SAT): share route decidability and clarify gadget APIs #1191

Closed

Description

@edemaine

Address the concrete nonblocking nits from #1174's outside review, based after #1180 (and the counting polish follow-up if needed). Preserve the approved #1147/#1174 heads.

Scope: one reusable orthogonal-route decidability instance replacing six private copies; crossover uniqueness source credit; list the paired crossover's headline iff/unique-extension results; remove the paired-geometry wrapper around the shared boundary theorem and update its clients. Keep the generic one-way DirectedWire API, which the reviewer explicitly distinguishes from paired equality wires.

Worker t3-96187517-polish owns this task on feat/parsimony-geometry-polish. Full validation, source-credit verification, abstract assessment and independent coordinator review are required before integration. These are follow-up improvements, not merge blockers for approved layers.

Activity

  1. self-assigned this
    on Oct 26, 2026
  2. edemaine commented on Oct 26, 2026

    @edemaine
    CollaboratorAuthor

    active — t3-96187517-polish owns the route-decidability and gadget-credit follow-up. It will follow the counting polish from #1190 on feat/parsimony-geometry-polish; parent owns implementation and validation. No changes to the approved lower heads or their merge eligibility are intended.

  3. edemaine commented on Oct 26, 2026

    @edemaine
    CollaboratorAuthor

    Worker t3-96187517-polish: active. Shared decidability and documentation cleanup implemented; full validation is running before the follow-up PR. Approved #1147/#1174 heads remain unchanged.

  4. edemaine commented on Oct 26, 2026

    @edemaine
    CollaboratorAuthor

    Worker t3-96187517-polish: awaiting review of #1196 at 27d5220, based on #1195. All six local gates pass; source credit and all six replaced deciders were checked. Outside-fleet coordinator review requested. Validation and no-impact abstract assessment are recorded in the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions