Skip to content

Implement Figure.pygmtlogo for the PyGMT logo#4616

Merged
seisman merged 31 commits into
mainfrom
pygmtlogo
Jul 4, 2026
Merged

Implement Figure.pygmtlogo for the PyGMT logo#4616
seisman merged 31 commits into
mainfrom
pygmtlogo

Conversation

@seisman

@seisman seisman commented May 2, 2026

Copy link
Copy Markdown
Member

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 and issues

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

github-actions Bot commented May 2, 2026

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
added pygmt/tests/baseline/test_pygmtlogo_design_circle-horizontal.png
added pygmt/tests/baseline/test_pygmtlogo_design_circle-vertical.png
added pygmt/tests/baseline/test_pygmtlogo_design_hexagon-horizontal.png
added pygmt/tests/baseline/test_pygmtlogo_design_hexagon-vertical.png
added pygmt/tests/baseline/test_pygmtlogo_wordmark_horizontal_circle.png
added pygmt/tests/baseline/test_pygmtlogo_wordmark_horizontal_hexagon.png
added pygmt/tests/baseline/test_pygmtlogo_wordmark_none_circle.png
added pygmt/tests/baseline/test_pygmtlogo_wordmark_none_hexagon.png
added pygmt/tests/baseline/test_pygmtlogo_wordmark_vertical_circle.png
added pygmt/tests/baseline/test_pygmtlogo_wordmark_vertical_hexagon.png

Image diff(s)

Details

Added images

  • 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_horizontal_circle.png

  • test_pygmtlogo_wordmark_horizontal_hexagon.png

  • test_pygmtlogo_wordmark_none_circle.png

  • test_pygmtlogo_wordmark_none_hexagon.png

  • test_pygmtlogo_wordmark_vertical_circle.png

  • test_pygmtlogo_wordmark_vertical_hexagon.png

Modified images

Path Old New

Report last updated at commit 2197015

…_pygmtlogo_circle_no_wordmark (#4617)

Co-authored-by: Yvonne Fröhlich <94163266+yvonnefroehlich@users.noreply.github.com>
…4634)

Co-authored-by: Yvonne Fröhlich <94163266+yvonnefroehlich@users.noreply.github.com>
Comment thread pygmt/src/pygmtlogo.py
Comment on lines +66 to +67
"light": (color_light, blue, color_dark),
"dark": (color_dark, yellow, color_light),

@yvonnefroehlich yvonnefroehlich Jun 18, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
"light": (color_light, blue, color_dark),
"dark": (color_dark, yellow, color_light),
"light": (color_light, blue, red),
"dark": (color_dark, yellow, red),

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

For the color choices, I feel the old version is better, because the GMT-red color makes the text “GMT” too visually heavy and eye-catching, drawing attention away from the logo symbol.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hm, that's true coloring also "GMT" makes the wordmark more prominent. I am fine with both versions. Let's see what the others think / prefer 🙂.

@seisman

seisman commented Jun 24, 2026

Copy link
Copy Markdown
Member Author

Considering that the paper is currently under the “Awaiting Editor Decision” status and will likely be accepted without another round of review, the next step will be proofreading, which will be our last opportunity to make any changes to the manuscript. If everything goes well, I expect the paper to be accepted within a few days and enter the proofreading stage in about two weeks.

In other words, we need to finalize the logos and make the v0.19.0 release within the next two weeks. I think we should decide whether there are any further changes needed for the logo designs. If everything looks good, we can ask the other maintainers for final reviews. If there are still choices to be made (e.g., #4616 (comment)), we can ask the other maintainers to vote on the remaining options.

@seisman

seisman commented Jun 29, 2026

Copy link
Copy Markdown
Member Author

@GenericMappingTools/pygmt-maintainers I think this PR is ready for final review.

In summary, it implements 24 variants of the PyGMT logo. You can preview all of the proposed designs here:
#4616 (comment) (Click "Details" in the "Image Diff" section).

The only remaining point under discussion is the color of the wordmark, i.e., whether the “GMT” letters should be black/white or red (see #4616 (comment), #4627 (comment), and #4616 (comment)).

Since we’d like to finalize the PyGMT logo for the upcoming v0.19.0 release, which will accompany the PyGMT paper, we’d really appreciate your feedback on the proposed designs and, in particular, your preference for the wordmark color.

@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 Jun 29, 2026
@michaelgrund

Copy link
Copy Markdown
Member

Sorry for answering late but my time is currently very limited due to parental leave. Potentially it's getting better in late July. Everything looks good to me, I really like that a logo can be used in various formats and colors very soon!

Comment thread pygmt/src/pygmtlogo.py
@seisman seisman merged commit b6c4820 into main Jul 4, 2026
23 of 24 checks passed
@seisman seisman deleted the pygmtlogo branch July 4, 2026 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Brand new 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