Skip to content

Let grdclip handle polar resampling via -Ar|t#5264

Merged
PaulWessel merged 3 commits intomasterfrom
grdclip-polar
Jun 6, 2021
Merged

Let grdclip handle polar resampling via -Ar|t#5264
PaulWessel merged 3 commits intomasterfrom
grdclip-polar

Conversation

@PaulWessel
Copy link
Member

Inspired by #5260, do the same when creating mask grids from polygons if we wish to resample the input polygon along straight theta and r path segments. Merge after 6.2.0, and will have no effect until #5263 have been merged as well.

Inspired by #5260, do the same when creating mask grids from polygons if we wish to resample the input polygon along straight theta and r path segments.  Merge after 6.2.0, and will have no effect until #5263 have been merged as well.
@PaulWessel PaulWessel added the new feature PR that implements a new feature or capability in GMT label May 26, 2021
@PaulWessel PaulWessel added this to the Future release milestone May 26, 2021
@PaulWessel PaulWessel self-assigned this May 26, 2021
@PaulWessel PaulWessel changed the title WIP Let grdclip handle polar resampling via -Ar|t Let grdclip handle polar resampling via -Ar|t Jun 6, 2021
@PaulWessel
Copy link
Member Author

Sorry, I have no test for this but it is the same under-the-hood machinery as used in plot and plot3d, so it is expected to work as is, @seisman.

@PaulWessel PaulWessel merged commit 434b1b4 into master Jun 6, 2021
@PaulWessel PaulWessel deleted the grdclip-polar branch June 6, 2021 05:00
@maxrjones maxrjones added new core module feature PR that implements a new core module feature add-changelog Add PR to the changelog labels Jun 8, 2021
@seisman seisman removed this from the Future release milestone Jan 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

add-changelog Add PR to the changelog new core module feature PR that implements a new core module feature new feature PR that implements a new feature or capability in GMT

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants