Prune rejected A03 moves and scan ordered via occupancy runs - #113
Open
seveibar wants to merge 3 commits into
Open
Prune rejected A03 moves and scan ordered via occupancy runs#113seveibar wants to merge 3 commits into
seveibar wants to merge 3 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
A03 spends substantial search time evaluating moves that are already closed or cannot improve their best known cost, and scanning empty or duplicate occupancy cells for through-vias. Prune those rejected moves using an admissible lower bound and add ordered occupancy runs for via queries. The original cost arithmetic, owner order, queue ordering and search budgets are preserved.
Direct callers keep dense queries by default; the autorouter integration explicitly selects
viaOccupantQuery: "owner-runs". This is an algorithm/data-structure change with no query-result cache, native routing engine or parallel routing.This PR freezes the benchmarked dependency commit
48c1ec6dfa61d83d58f8e3b11b97701918147c78. Paired with autorouter commitfb671fbdacaaf822ddf19b44eba953282a7101a6, fresh serial Blacksmith runs measured:That is 28.02% less stage time across these two diagnostic boards, not a full-corpus performance result or a 2× claim. Runs used the same Ampere-1a Blacksmith machine and Bun 1.4.2, baseline-first on board 2 and candidate-first on board 6. Complete output bytes, completion status and iteration counts match. Actual source and installed dependency bytes were verified before and after the runs.
Validation: