Skip to content

Add clarification to GridQubit neighbors function#7521

Merged
dstrain115 merged 2 commits intoquantumlib:mainfrom
dstrain115:grid_neighbors_docs
Jul 21, 2025
Merged

Add clarification to GridQubit neighbors function#7521
dstrain115 merged 2 commits intoquantumlib:mainfrom
dstrain115:grid_neighbors_docs

Conversation

@dstrain115
Copy link
Copy Markdown
Collaborator

  • An internal bug noted that there could be confusion if you assumed that GridQubit.neighbors() returned only the neighbors on a specific device layout.
  • This changes only the documentation to point out that this function is device-agnostic.

- An internal bug noted that there could be confusion if
you assumed that GridQubit.neighbors() returned only the
neighbors on a specific device layout.
- This changes only the documentation to point out that this
function is device-agnostic.
@dstrain115 dstrain115 requested review from a team and vtomole as code owners July 21, 2025 13:34
@github-actions github-actions bot added the size: S 10< lines changed <50 label Jul 21, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Jul 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.68%. Comparing base (c434f94) to head (d1aaf58).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7521      +/-   ##
==========================================
- Coverage   98.68%   98.68%   -0.01%     
==========================================
  Files        1091     1091              
  Lines       96948    96948              
==========================================
- Hits        95674    95673       -1     
- Misses       1274     1275       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Comment thread cirq-core/cirq/devices/grid_qubit.py Outdated
Co-authored-by: Pavol Juhas <pavol.juhas@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: S 10< lines changed <50

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants