Skip to content

Enable Lazy Execution for Sentinel-1 GRD Analysis Mode#85

Merged
konstntokas merged 9 commits into
mainfrom
konstntokas-xxx-bug_fix_sen1_grd
Jul 10, 2026
Merged

Enable Lazy Execution for Sentinel-1 GRD Analysis Mode#85
konstntokas merged 9 commits into
mainfrom
konstntokas-xxx-bug_fix_sen1_grd

Conversation

@konstntokas

Copy link
Copy Markdown
Collaborator
  • Sentinel-1 GRD analysis mode is now fully lazy, enabling seamless execution on
    local and distributed Dask clusters.

@codecov

codecov Bot commented Jul 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
xarray_eopf/amodes/sentinel1.py 100.00% <100.00%> (ø)
xarray_eopf/backend.py 100.00% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@pont-us
pont-us self-requested a review July 8, 2026 13:46
Comment thread xarray_eopf/amodes/sentinel1.py Outdated
Comment thread xarray_eopf/amodes/sentinel1.py Outdated
Comment thread xarray_eopf/backend.py Outdated
when not provided.
apply_rtc: Whether to apply radiometric terrain correction (RTC) for
Sentinel-1 analysis mode. Defaults to `True`.
cache_uri: Temporary path used to store intermediate results from the

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be good to clarify that this parameter is interpreted as an fsspec "urlpath". Just "URI" and "path" doesn't make explicit what values are allowed.

konstntokas and others added 3 commits July 10, 2026 08:12
Co-authored-by: Pontus Lurcock <pont@talvi.net>
Co-authored-by: Pontus Lurcock <pont@talvi.net>
@konstntokas
konstntokas merged commit 39fa42e into main Jul 10, 2026
2 checks passed
@konstntokas
konstntokas deleted the konstntokas-xxx-bug_fix_sen1_grd branch July 10, 2026 06:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants