Skip to content

[na] Let movement weight change the chosen schedule - #535

Merged
cursor[bot] merged 1 commit into
mainfrom
cursor/movement-weight-choice-aa1b
Oct 1, 2026
Merged

cursor[bot] merged 1 commit into
mainfrom
cursor/movement-weight-choice-aa1b

Conversation

@arniber21

@arniber21 arniber21 commented Oct 1, 2026 •

Copy link
Copy Markdown
Owner

Summary

Stage, idle, and transfer weights stay fixed and positive. movement_time_weight then changes which verified one-gate orientation is chosen.

With idle and transfer both at 1, the stationary partner's idle swamps the two-transfer savings, so the shorter move wins at every non-negative movement weight. The fixed transfer weight sits just above that idle cost. A movement weight below the longer move's extra time keeps the one-atom orientation. A movement weight above it takes the shorter two-atom move. Each verified total is lower under the weights that chose it.

Refs #484

Test

cargo test -p quon_na --features mlir --lib -- movement_time_weight_selects

Open in Web Open in Cursor 

Stage, idle, and transfer stay positive and fixed. The fixed transfer
weight outweighs the stationary partner's idle, so a low movement
weight keeps the longer one-atom move and a high movement weight takes
the shorter two-atom move. Each verified total is lower under the
weights that chose it.

Co-authored-by: Arnab Ghosh <ag2583@cornell.edu>
@vercel

vercel Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
quon Ready Ready Preview Oct 1, 2026 10:02am UTC

@cursor
cursor Bot marked this pull request as ready for review October 1, 2026 10:06
@cursor
cursor Bot marked this pull request as draft October 1, 2026 10:08
@cursor
cursor Bot marked this pull request as ready for review October 1, 2026 10:09
@cursor
cursor Bot merged commit 6f961f5 into main Oct 1, 2026
22 of 23 checks passed

This branch was successfully deployed

1 active deployment
Preview — 2cf07748 Deployed Oct 1, 2026 by vercel[bot]
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