Skip to content

test: add deterministic overlap regression coverage for issue #12#44

Open
CuboYe wants to merge 2 commits intotscircuit:mainfrom
CuboYe:tez/followup-issue12-regression
Open

test: add deterministic overlap regression coverage for issue #12#44
CuboYe wants to merge 2 commits intotscircuit:mainfrom
CuboYe:tez/followup-issue12-regression

Conversation

@CuboYe
Copy link
Copy Markdown

@CuboYe CuboYe commented Mar 15, 2026

/claim #12

Summary

Adds focused follow-up test coverage for the issue #12 lane by asserting deterministic, overlap-free pipeline output across repeated runs of the same circuit input.

What changed

  • Added LayoutPipelineSolver02 deterministic overlap-free output across repeated runs test in:
    • tests/LayoutPipelineSolver/LayoutPipelineSolver02.test.ts
  • New test:
    • runs LayoutPipelineSolver 3 times on the same ExampleCircuit02 input,
    • asserts zero overlaps each run,
    • asserts deterministic placement consistency for representative chip U1.

Validation

  • bun test tests/LayoutPipelineSolver/LayoutPipelineSolver02.test.ts
  • Result: 5 pass, 1 skip, 0 fail

Why this helps

This gives maintainers concrete regression protection for the issue #12 class (layout quality/determinism) without duplicating broader implementation already in-flight in #39.

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 15, 2026

@QuinnYates is attempting to deploy a commit to the tscircuit Team on Vercel.

A member of the Team first needs to authorize it.

@CuboYe

This comment was marked as spam.

@CuboYe

This comment was marked as spam.

@CuboYe

This comment was marked as spam.

@CuboYe

This comment was marked as spam.

@CuboYe

This comment was marked as spam.

@CuboYe

This comment was marked as spam.

@CuboYe

This comment was marked as spam.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants