Skip to content

add debug labels to NetLabelPlacementSolver#208

Merged
seveibar merged 4 commits intomainfrom
add_debug_labels_netlabelplacement_solver
Apr 22, 2026
Merged

add debug labels to NetLabelPlacementSolver#208
seveibar merged 4 commits intomainfrom
add_debug_labels_netlabelplacement_solver

Conversation

@Sang-it
Copy link
Copy Markdown
Contributor

@Sang-it Sang-it commented Apr 22, 2026

No description provided.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 22, 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 Apr 22, 2026 11:33pm

Request Review

Copy link
Copy Markdown
Member

@imrishabh18 imrishabh18 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add a test with an example snapshot?

@seveibar seveibar changed the title chore: add add debug labels to netlabelplacement solver add add debug labels to netlabelplacement solver Apr 22, 2026
Copy link
Copy Markdown
Contributor

@seveibar seveibar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Careful w/ AI PR titles, your original title had "chore:", these designations are often wrong. We don't care about commit names or branch names but we DO care about PR titles a lot because they go into a changelog and attribution system that is scanned by AI

The main reason for my request for changes is you're violating "variable transparency"- make sure the labels reflect the actual data without renaming anything, this clarifies things greatly see my comments for examples...

Comment thread lib/solvers/NetLabelPlacementSolver/NetLabelPlacementSolver.ts Outdated
Comment thread lib/solvers/NetLabelPlacementSolver/NetLabelPlacementSolver.ts Outdated
Comment thread lib/solvers/NetLabelPlacementSolver/NetLabelPlacementSolver.ts Outdated
@Sang-it Sang-it changed the title add add debug labels to netlabelplacement solver add debug labels to NetLabelPlacementSolver Apr 22, 2026
@seveibar seveibar merged commit ba21c63 into main Apr 22, 2026
5 checks passed
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.

3 participants