Skip to content

pygmt.grdclip: Parameters 'between' and 'replace' accept a 2-D sequence #3883

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

seisman
Copy link
Member

@seisman seisman commented Mar 29, 2025

@seisman seisman added the enhancement Improving an existing feature label Mar 29, 2025
@seisman seisman added this to the 0.16.0 milestone Mar 29, 2025
@seisman seisman changed the title pygmt.grdclip: Parameters 'between' and 'new' now can accept a 2-D sequence pygmt.grdclip: Parameters 'between' and 'replace' accept a 2-D sequence Mar 31, 2025
@seisman seisman marked this pull request as ready for review March 31, 2025 15:26
@seisman seisman added needs review This PR has higher priority and needs review. and removed needs review This PR has higher priority and needs review. labels Mar 31, 2025
@seisman seisman removed this from the 0.16.0 milestone Apr 1, 2025
@seisman seisman marked this pull request as draft April 2, 2025 04:10
seisman and others added 2 commits April 2, 2025 12:13
Co-authored-by: Michael Grund <23025878+michaelgrund@users.noreply.github.com>
@seisman seisman marked this pull request as ready for review April 3, 2025 05:22
@seisman seisman added this to the 0.16.0 milestone Apr 3, 2025
@seisman seisman added the needs review This PR has higher priority and needs review. label Apr 3, 2025
@seisman seisman added final review call This PR requires final review and approval from a second reviewer and removed needs review This PR has higher priority and needs review. labels Apr 21, 2025
@seisman seisman requested a review from a team April 21, 2025 04:56
Co-authored-by: Yvonne Fröhlich <94163266+yvonnefroehlich@users.noreply.github.com>
@seisman
Copy link
Member Author

seisman commented Apr 22, 2025

After changes in this PR, the API docs only list two aliases, for -R and -V:
image

That's also why I opened PR #3881.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improving an existing feature final review call This PR requires final review and approval from a second reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants