unwrap(whirlwind): expose the options added since the backend landed

[scottstanie]

WhirlwindOptions has not kept up with whirlwind's own API, so several knobs have been unreachable from a dolphin config. This adds them, each keeping whirlwind's own default, so behavior is unchanged unless a config sets one.

Connected components

Components are now always grown by whirlwind's SNAPHU-style ambiguity-wiggle algorithm with ThickenCosts on, which is what production SNAPHU does. That is pinned at the call site rather than left to whirlwind's defaults, so a change upstream cannot silently alter dolphin's labels, and the algorithm is deliberately not a config option.

conncomp_min_coherence and conncomp_reliability are new and tune how conservatively that grow cuts.

This does mean the pre-existing cost_threshold, conncomp_sigma and conncomp_cycle_prob have nothing left to tune — they only ever fed whirlwind's legacy coherence-cost labeling, which dolphin no longer selects. They are kept on the model so existing configs still load, and documented as unused. Happy to delete them outright instead if a breaking config change is acceptable.

Phase solve

• bridge — give disjoint components a consistent integer cycle.
• connect_gaps / connect_gaps_max_px — draw phase paths across bounded runs of invalid pixels before the solve, then drop those synthetic pixels from both outputs, so the solver can pick a relative 2π level between regions that would otherwise be solved independently. Off by default. This is what levels NISAR's fixed-PRF sub-swaths.
• goldstein_alpha / goldstein_psize — whirlwind's internal Goldstein filter.

Double pre-processing fix

whirlwind runs Goldstein filtering and PS interpolation itself, in Rust. run_goldstein / run_interpolation were running dolphin's Python pre-process and leaving whirlwind's own pass available, so a config that asked for Goldstein filtered the same interferogram twice. Those requests now route into whirlwind's versions instead, with preprocess_options.alpha taking precedence over goldstein_alpha.

Notes

• Requires whirlwind >= 0.10.0 for connect_gaps. whirlwind remains an optional dependency (imported inside the function), so nothing changes for installs without it.
• TestWhirlwind now asserts what actually reaches ww.unwrap, rather than only that the output files exist.
• The red ubuntu-latest / macos-14 checks are a pre-existing CI break on main (ModuleNotFoundError: No module named 'rich' while importing tests/conftest.py), not from this PR — the same failure is on Bump actions/setup-python from 6 to 7 in the actions group across 1 directory #737, Add Gaussian multilooking option for phase linking #734, and on an unrelated run from 2026-09-07.

🤖 Generated with Claude Code

https://claude.ai/code/session_01GZgYKTv2qVefPeX9S7a6BN