Skip to content

Fix zero-length trace path cleanup#387

Open
ar-amk wants to merge 1 commit into
tscircuit:mainfrom
ar-amk:codex/tscircuit-78-simplify-path-dedup
Open

Fix zero-length trace path cleanup#387
ar-amk wants to merge 1 commit into
tscircuit:mainfrom
ar-amk:codex/tscircuit-78-simplify-path-dedup

Conversation

@ar-amk
Copy link
Copy Markdown

@ar-amk ar-amk commented May 20, 2026

/claim #78

Summary

  • strip consecutive duplicate path points before collinear cleanup so zero-length trace segments cannot survive post-processing
  • rerun collinear collapse with duplicate-point cleanup until the path stops shrinking
  • add focused unit coverage for duplicate points and zero-length two-point paths

Tests

  • bun test tests/solvers/TraceCleanupSolver/simplifyPath.test.ts tests/solvers/TraceCleanupSolver/TraceCleanupSolver.test.ts
  • bun test
  • bun run format:check
  • bun run build
  • git diff --check

@vercel
Copy link
Copy Markdown

vercel Bot commented May 20, 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 May 20, 2026 10:12pm

Request Review

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.

1 participant