Skip to content

Add repro for too many candidates in SingleTargetNecessaryCrampedPortPointSolver#640

Merged
0hmX merged 2 commits intomainfrom
add-repro-cramed-port-expess-candidates-2
Mar 10, 2026
Merged

Add repro for too many candidates in SingleTargetNecessaryCrampedPortPointSolver#640
0hmX merged 2 commits intomainfrom
add-repro-cramed-port-expess-candidates-2

Conversation

@0hmX
Copy link
Copy Markdown
Contributor

@0hmX 0hmX commented Mar 10, 2026

  • Add test for necessary cramped port point solver to prevent duplicate candidate explosion
  • Refactor necessary cramped port point solver test to remove unused statistics and ensure candidate count exceeds 500

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 10, 2026

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

Project Deployment Actions Updated (UTC)
capacity-node-autorouter Ready Ready Preview, Comment Mar 10, 2026 1:30pm

Request Review

@tscircuitbot
Copy link
Copy Markdown

🏃 Benchmark This PR

Run benchmarks by commenting on this PR:

/benchmark [solver-name|all] [scenario-limit] --concurrency <n> --effort <n>

Examples:

  • /benchmark -> AutoroutingPipelineSolver, all scenarios (default concurrency uses the benchmark runner CPU count)
  • /benchmark AutoroutingPipelineSolver -> one solver, all scenarios
  • /benchmark all 20 -> all solvers, first 20 scenarios
  • /benchmark AutoroutingPipelineSolver 20 --concurrency 8 -> one solver, 20 scenarios, 8 workers
  • /benchmark AutoroutingPipelineSolver 20 --effort 2 -> one solver, 20 scenarios, 2x effort

Any PR whose title contains [BENCHMARK TEST] will automatically run the benchmark workflow on PR updates.

@0hmX 0hmX merged commit a646289 into main Mar 10, 2026
14 checks passed
@0hmX 0hmX deleted the add-repro-cramed-port-expess-candidates-2 branch March 10, 2026 13:50
@tscircuitbot
Copy link
Copy Markdown


Thank you for your contribution! 🎉

PR Rating: ⭐⭐⭐
Impact: Major

Track your contributions and see the leaderboard at: tscircuit Contribution Tracker


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