Subgrid shift

[WanHsuanLin]

1. Add implementation for subgrid_shift_x and subgrid_shift_y as described in Shift subset of positions defined by a list of indices. #31
2. Add assertion to check if the shift operation causes column/row order changes
3. Can take ilist.IList and slice as input for column/row indices
4. Add test cases.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[Copilot]

Pull Request Overview

This PR implements subgrid shift functionality for Grid objects, allowing selective shifting of columns and rows while preserving grid integrity. The implementation adds methods to shift specific column or row indices by a given amount while automatically adjusting spacing to maintain grid structure.

• Adds shift_subgrid_x and shift_subgrid_y methods with support for both ilist.IList and slice inputs
• Implements validation to prevent invalid shifts that would change column/row ordering
• Includes comprehensive test coverage for various scenarios including edge cases

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
src/bloqade/geometry/dialects/grid/types.py	Core implementation of subgrid shift methods with spacing adjustment logic
src/bloqade/geometry/dialects/grid/stmts.py	Statement definitions for the new shift operations
src/bloqade/geometry/dialects/grid/concrete.py	Concrete interpreter implementations for subgrid shift statements
src/bloqade/geometry/dialects/grid/_interface.py	Public API interface definitions with documentation
src/bloqade/geometry/dialects/grid/init.py	Module exports for new shift operations
test/grid/test_types.py	Comprehensive test cases covering various input types and edge cases
test/grid/test_concrete.py	Integration tests for concrete interpreter functionality

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[weinbe58]

LGTM, just make sure to run pre-commit install and pre-commit run --all-files to get the linting CI to pass

[weinbe58]

closes #33

[weinbe58]

Since this is a user input we should make this an exception instead of an assertion. Generally Assertions are used to enforce internal logic inside a code to catch potential bugs while exceptions are indicating invalid inputs from a user.