From 152056841bff26618b32d8df6c0e6b8bde936667 Mon Sep 17 00:00:00 2001 From: Claude Date: Wed, 20 May 2026 12:24:07 +0000 Subject: [PATCH] fix(dolphin): swap default half_window to (5, 11) for (y, x) order The previous default (11, 5) was inverted relative to the (y, x) ordering documented on the field. OPERA CSLC-S1 has y-spacing 10 m and x-spacing 5 m, so a square window in meters needs ~2x more pixels in x than y (matching the existing strides default of (6, 12) and legacy sweets' y=7, x=14). (11, 5) instead produced a 230 m x 55 m window. --- src/sweets/_dolphin.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/sweets/_dolphin.py b/src/sweets/_dolphin.py index 66fab84..30ac1b3 100644 --- a/src/sweets/_dolphin.py +++ b/src/sweets/_dolphin.py @@ -62,10 +62,11 @@ class DolphinOptions(BaseModel): """ half_window: tuple[int, int] = Field( - default=(11, 5), + default=(5, 11), description=( - "Half-window (y, x) for phase linking. The default of (11, 5) gives" - " roughly square windows for the OPERA 10x5 m geocoded posting." + "Half-window (y, x) for phase linking. The default of (5, 11) gives" + " roughly square windows for the OPERA 10x5 m geocoded posting" + " (y-spacing 10 m, x-spacing 5 m)." ), ) strides: tuple[int, int] = Field(