Skip to content

Conversation

seisman
Copy link
Member

@seisman seisman commented Sep 17, 2025

No description provided.

@seisman seisman marked this pull request as draft September 17, 2025 14:04
@seisman seisman changed the base branch from main to sequence_join/datetime September 17, 2025 14:04
@seisman seisman added maintenance Boring but important stuff for the core devs skip-changelog Skip adding Pull Request to changelog labels Sep 18, 2025
@seisman seisman added this to the 0.17.0 milestone Sep 18, 2025
@seisman seisman mentioned this pull request Sep 18, 2025
39 tasks
Base automatically changed from sequence_join/datetime to main September 23, 2025 05:55
@seisman seisman marked this pull request as ready for review September 23, 2025 07:59
@seisman seisman added the needs review This PR has higher priority and needs review. label Sep 23, 2025
case "J":
alias = Alias(value, name="projection")
case "R":
alias = Alias(value, name="region", sep="/", size=(4, 6))
Copy link
Member Author

Choose a reason for hiding this comment

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

The size is usually 4, but can be 6 for 3-D plots.

>>> landmask = pygmt.grdlandmask(spacing=1, region=[125, 130, 30, 35])
"""
if kwargs.get("I") is None or kwargs.get("R") is None:
if kwargs.get("I") is None or kwargs.get("R", region) is None:
Copy link
Member Author

Choose a reason for hiding this comment

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

Check if region/R is not specified.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Boring but important stuff for the core devs needs review This PR has higher priority and needs review. skip-changelog Skip adding Pull Request to changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant