Yh cyber windowed blending sampler

[yh-cyber]

Summary

Validation

• scripts/quality-check.sh passes locally
• Appropriate /test commands were run and are passing (e.g. /test gpu)
• Unit tests were added / e2e tests were added where applicable
• Manual testing, if applicable (describe further): added test_sampler file to add unit tests, all pass

Checklist

• The change is focused and easy to review.
• Tests were added or updated for behaviour changes.
• No secrets, generated files, or files that only make sense on my machine are included.
• Relevant documentation is updated.

Additional Comments

I think there might be something wrong with scripts/quality-check.sh for me.

This is what I got when I ran scripts/quality-check.sh

$ scripts/quality-check.sh==> Checking formatting (ruff format --check)17 files already formatted==> Checking code style (ruff check)error: a value is required for '[FILES]...' but none was suppliedFor more information, try '--help'.Found style problems. Many can be fixed automatically with:
scripts/format-check.sh --fix(terrain-diffusion) $ uv run ruff check src testsAll checks passed!(terrain-diffusion) 

[coveralls]

Coverage Report for CI Build 32797978609

Coverage increased (+1.0%) to 97.822%

Details

• Coverage increased (+1.0%) from the base build.
• Patch coverage: 162 of 162 lines across 2 files are fully covered (100%).
• No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.

---

Coverage Stats

Relevant Lines:	505
Covered Lines:	494
Line Coverage:	97.82%
Coverage Strength:	0.98 hits per line

---

💛 - Coveralls

[KurbyDoo]

overall the structure looks good, just some things to clean up function boundaries and make testing easier

[KurbyDoo]

I have a fix for the script error? Could you take a look #54

[yh-cyber]

I've implemented all changes!

[KurbyDoo]

Right now the windows are dependent on the region which is fine for now, later we will work to make them independent of the requested region and instead based on a global coordinate space. I'll make tickets for that later

[KurbyDoo]

thanks for addressing my comments, looks good to merge after this

[yh-cyber]

np, thnx. once you take a look and resolve the conversations, ill squash and merge.