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

@seismanseisman commented May 2, 2026

Copy link
Copy Markdown
Member

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

Logo implementation status

VariantColored + LightColored + DarkGrayscale + LightGrayscale + 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>
@seismanseisman added this to the 0.19.0 milestone May 2, 2026
@seismanseisman added the feature Brand new feature label May 2, 2026
@github-actions

github-actionsBot 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

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

PathOldNew

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>
@seisman

Copy link
Copy Markdown
MemberAuthor

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

Copy link
Copy Markdown
MemberAuthor

@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.

@seismanseisman 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 threadpygmt/src/pygmtlogo.py
@seisman
seisman merged commit b6c4820 into mainJul 4, 2026
23 of 24 checks passed
@seisman
seisman deleted the pygmtlogo branch July 4, 2026 15:41
@seismanseisman mentioned this pull request Jul 7, 2026
@seismanseisman removed the final review call This PR requires final review and approval from a second reviewer label Jul 23, 2026
@seismanseisman changed the title Implement Figure.pygmtlogo for the PyGMT logoFigure.pygmtlogo: Plot PyGMT logosJul 23, 2026
@seismanseisman changed the title Figure.pygmtlogo: Plot PyGMT logosImplement Figure.pygmtlogo for the PyGMT logoJul 27, 2026
@seismanseisman mentioned this pull request Jul 27, 2026
12 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

featureBrand new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@seisman@yvonnefroehlich@michaelgrund