Skip to content

Figure.pygmtlogo: Extend the region to get rid of the 'no_clip' parameter#4687

Open
seisman wants to merge 1 commit into
pygmtlogofrom
pygmtlogo-noclip
Open

Figure.pygmtlogo: Extend the region to get rid of the 'no_clip' parameter#4687
seisman wants to merge 1 commit into
pygmtlogofrom
pygmtlogo-noclip

Conversation

@seisman

@seisman seisman commented Jun 21, 2026

Copy link
Copy Markdown
Member

Extending the region parameter by 20% so that we no longer need the no_clip parameter when plotting hexagonal logos and wordmarks.

Changes this PR don't affect the logo appearance, but affect the debugging basemap in the tests for logo designs.

@seisman seisman added this to the 0.19.0 milestone Jun 21, 2026
@seisman seisman added maintenance Boring but important stuff for the core devs skip-changelog Skip adding Pull Request to changelog labels Jun 21, 2026
@seisman seisman marked this pull request as ready for review June 21, 2026 03:06
@seisman seisman requested a review from yvonnefroehlich June 21, 2026 03:06
@github-actions

Copy link
Copy Markdown
Contributor

Summary of changed images

This is an auto-generated report of images that have changed on the DVC remote

Status Path
modified pygmt/tests/baseline/test_pygmtlogo_design_circle-horizontal.png
modified pygmt/tests/baseline/test_pygmtlogo_design_circle-vertical.png
modified pygmt/tests/baseline/test_pygmtlogo_design_hexagon-horizontal.png
modified pygmt/tests/baseline/test_pygmtlogo_design_hexagon-vertical.png
modified pygmt/tests/baseline/test_pygmtlogo_wordmark_vertical_circle.png

Image diff(s)

Details

Added images

Modified images

Path Old New
test_pygmtlogo_design_circle-horizontal.png
test_pygmtlogo_design_circle-vertical.png
test_pygmtlogo_design_hexagon-horizontal.png
test_pygmtlogo_design_hexagon-vertical.png
test_pygmtlogo_wordmark_vertical_circle.png

Report last updated at commit f8d2a9c

@seisman seisman added the final review call This PR requires final review and approval from a second reviewer label Jun 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

final review call This PR requires final review and approval from a second reviewer maintenance Boring but important stuff for the core devs skip-changelog Skip adding Pull Request to changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant