Skip to content

Implement Figure.pygmtlogo for the PyGMT logo#4616

Draft
seisman wants to merge 1 commit intomainfrom
pygmtlogo
Draft

Implement Figure.pygmtlogo for the PyGMT logo#4616
seisman wants to merge 1 commit intomainfrom
pygmtlogo

Conversation

@seisman
Copy link
Copy Markdown
Member

@seisman seisman commented May 2, 2026

This PR tracks the implementation of the PyGMT logos in different versions.

Logo implementation status

Variant Colored + Light Colored + Dark Grayscale + Light Grayscale + Dark
Circular, no wordmark
Circular, horizontal wordmark
Circular, vertical wordmark
Hexagon, no wordmark
Hexagon, horizontal wordmark
Hexagon, vertical wordmark

Related PRs

Hexagon shape

Pro

  • Seems to be quite popular (even for Python)

Contra

  • lost of context / connection to Earth and compass
  • difficult of implement based on fractions rather than on empircal factors
  • difficult to make to logo conistent across both shapes
  • 12 variants already for circle shape (color/bw, no/hor./vert. wordmark, light/dark), multiple tests with baseline images
  • longer code, difficult to maintain

Some important comments from #3849

Known issues

  • After plotting the logo, -X+w is expected to shift the plotting origin in the x-direction by logo width, but it actually shifts more than that. It's likely an issue with the image module.

…GMT logo (#3849)

Co-authored-by: Dongdong Tian <seisman.info@gmail.com>
Co-authored-by: Michael Grund <23025878+michaelgrund@users.noreply.github.com>
Co-authored-by: Wei Ji <23487320+weiji14@users.noreply.github.com>
@seisman seisman added this to the 0.19.0 milestone May 2, 2026
@seisman seisman added the feature Brand new feature label May 2, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 2, 2026

Summary of changed images

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

Status Path
added pygmt/tests/baseline/test_pygmtlogo.png

Image diff(s)

Details

Added images

  • test_pygmtlogo.png

Modified images

Path Old New

Report last updated at commit 4868c8a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Brand new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants