Skip to content

Conversation

@seisman
Copy link
Member

@seisman seisman commented Dec 15, 2025

Related to an pstream GMT bug originally found in GenericMappingTools/pygmt-paper-figures#17 (comment) and fixed in GenericMappingTools/gmt#8838.

This PR adds a workaround for it and fix the baseline image for the test.

…ane is set to True or facede_fill/facede_pen is set.
@seisman seisman added this to the 0.18.0 milestone Dec 15, 2025
@seisman seisman added bug Something isn't working needs review This PR has higher priority and needs review. labels Dec 15, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 15, 2025

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_grdview_facadepen_default_plane.png

Image diff(s)

Details

Added images

Modified images

Path Old New
test_grdview_facadepen_default_plane.png

Report last updated at commit ef579dd

seisman and others added 4 commits December 16, 2025 05:49
Co-authored-by: Yvonne Fröhlich <94163266+yvonnefroehlich@users.noreply.github.com>
Co-authored-by: Yvonne Fröhlich <94163266+yvonnefroehlich@users.noreply.github.com>
@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 Dec 17, 2025
@seisman seisman requested a review from a team December 17, 2025 00:05
@seisman seisman removed the final review call This PR requires final review and approval from a second reviewer label Dec 17, 2025
@seisman seisman merged commit 37a2e44 into main Dec 17, 2025
24 checks passed
@seisman seisman deleted the grdview/facade_pen branch December 17, 2025 03:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants