[Copilot]

The test CI job failed in test_mask_aware_tile_positions_shift_per_row with ValueError: too many values to unpack (expected 2). _mask_aware_tile_positions now returns (x0_list, overlap_px, optimal_y0) 3-tuples, but the test still unpacked each run as a 2-tuple.

Changes

• tests/test_shell.py: unpack top and bottom runs as x0s, _, _ instead of x0s, _.

# before
[x0 for x0s, _ in top for x0 in x0s] == [148]
# after
[x0 for x0s, _, _ in top for x0 in x0s] == [148]

This is a test-only change. The production code is untouched, and the expected x offsets (148 and 268) are unchanged.

[codecov]

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

[Copilot]

🟢 Approval recommended

The test now correctly matches the function’s documented return type without changing behavior.

0 open findings

What changed in this PR

Updates a stale test to match _mask_aware_tile_positions’ three-value return structure.

Changes:

• Unpacks all three tuple values while preserving existing offset assertions.

File	Description
tests/​test_shell.py	Fixes tuple unpacking in the mask-aware positioning test.

🧠 Review effort: Balanced

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.