isce-framework / isce-framework/dolphin
Add option to skip wrapped phase processing over water given some water mask
Open
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 153
- Forks
- 33
- PR merge metrics
- No merged PRs in 30d
Description
Doesn't need to be the default, but we could save a bunch of time on some island frames if we conservatively skipped processing the ocean regions. I believe there are some frames where there's just one tiny land part that intersected a frame boundary:
- We would need to get a reliable water mask, turn it into a polygon with
gdal_polyonize
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No source files, tests, or processing entry points are named. Start by locating wrapped phase processing and determining how a reliable water mask could be supplied and converted with gdal_polygonize. Done means an optional, conservative water mask skips ocean regions without excluding the land needed for processing.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- performance
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100