Skip to content

Conversation

@gmgunter
Copy link
Member

Adds a new function isce3.geometry.get_bounding_radar_grid() that returns a RadarGridParameters object that spans an input GeoGridParameters object. The output radar grid is the smallest radar grid that contains all points within the input geocoded grid over a user-specified height range. Additional margin may be added to extend the radar grid azimuth & range extents.

This functionality is useful for implementing the static layers workflow. Many of the layers in the static layers product are computed from an input radar grid, but the input to the static layers workflow is a geo grid -- not a radar grid. Therefore we will need the ability to compute a radar grid that spans the input geo grid.

@gmgunter gmgunter added this to the R05.00.0 milestone Aug 16, 2025
@gmgunter gmgunter requested review from gshiroma and nemo794 August 16, 2025 00:10
@gmgunter
Copy link
Member Author

@gshiroma @nemo794 this is a copy of a PR that was approved on the internal GitHub server: https://github-fn.jpl.nasa.gov/isce-3/isce/pull/2097

Copy link
Contributor

@gshiroma gshiroma left a comment

Choose a reason for hiding this comment

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

LGTM. This PR was reviewed internally. Thank you, @gmgunter . Just make sure that the unit tests pass.

Copy link
Contributor

@nemo794 nemo794 left a comment

Choose a reason for hiding this comment

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

Ditto @gshiroma . Thanks, @gmgunter !

@gmgunter gmgunter merged commit 9c58871 into isce-framework:develop Aug 18, 2025
5 checks passed
@gmgunter gmgunter deleted the bounding-radar-grid branch August 18, 2025 17:22
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.

3 participants