Skip to content

fix: remove extra trace lines in post-processing (repro78)#124

Open
CuboYe wants to merge 1 commit intotscircuit:mainfrom
CuboYe:bounty/fix-78-tdd
Open

fix: remove extra trace lines in post-processing (repro78)#124
CuboYe wants to merge 1 commit intotscircuit:mainfrom
CuboYe:bounty/fix-78-tdd

Conversation

@CuboYe
Copy link

@CuboYe CuboYe commented Mar 3, 2026

Summary

  • add a focused repro test for issue Fix extra trace lines in post-processing step #78 using the DISCH fixture with schMaxTraceDistance=6
  • assert duplicate raw segments exist in solver traces (bug repro)
  • add post-processing trace line dedupe per globalConnNetId + segment

Verification

  • bun test tests/solvers/SchematicTracePipelineSolver/SchematicTracePipelineSolver_repro78.test.ts
  • bun test

Closes #78
/claim #78

@vercel
Copy link

vercel bot commented Mar 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
schematic-trace-solver Ready Ready Preview, Comment Mar 3, 2026 10:34pm

Request Review

@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.

@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.

Fix extra trace lines in post-processing step

2 participants