[na] Refuse entangle merges inside isolation spacing Refs #484 - #537
Merged
Merged
Conversation
Order-only compaction merged disjoint entangle layers without checking replayed positions. A weighted stage split could put non-partner atoms 15.62 µm apart in one cycle, inside the 18.75 µm isolation spacing. Refuse that merge and keep the split. Pairs outside isolation still merge. The default time objective is unchanged. Refs #484 Co-authored-by: Arnab Ghosh <ag2583@cornell.edu>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
arniber21
marked this pull request as ready for review
October 1, 2026 11:57
This branch was successfully deployed
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
2 participants
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.
Order-only compaction merged a weighted stage split into one entangle cycle whenever the atoms were disjoint. That merge skipped a Rydberg recheck at the entangle-cycle positions, so non-partner atoms one entanglement-grid diagonal apart (15.62 µm) could share a cycle inside the 18.75 µm isolation spacing.
Compaction now replays moves from the declared start and refuses that merge, leaving the split in place. Pairs outside isolation still merge. Static final-layout R2/R3 stays off on the order-only path. The default time objective is unchanged, and the qaoa resource-report golden still matches a verified schedule.
Refs #484