Skip to content

DOC/website: Move citiation information from README to separate file and subsection "citing" - #4784

Merged
seisman merged 35 commits into
mainfrom
add-subsection-citing
Aug 3, 2026
Merged

DOC/website: Move citiation information from README to separate file and subsection "citing"#4784
seisman merged 35 commits into
mainfrom
add-subsection-citing

Conversation

@yvonnefroehlich yvonnefroehlich added this to the 0.20.0 milestone Jul 30, 2026
@yvonnefroehlich yvonnefroehlich self-assigned this Jul 30, 2026
@yvonnefroehlich yvonnefroehlich added documentation Improvements or additions to documentation skip-changelog Skip adding Pull Request to changelog labels Jul 30, 2026
@yvonnefroehlich yvonnefroehlich changed the title DOC/website: Add subsection "citing" DOC/website: Add separate file and subsection "citing" Jul 30, 2026
@yvonnefroehlich yvonnefroehlich added the needs review This PR has higher priority and needs review. label Jul 30, 2026
Comment thread README.md Outdated
@yvonnefroehlich yvonnefroehlich changed the title DOC/website: Add separate file and subsection "citing" DOC/website: Move from citiation information from README to separate file and subsection "citing" Jul 30, 2026
Comment thread doc/citing.md Outdated
Comment thread doc/citing.md Outdated
Comment thread doc/citing.md Outdated
Comment thread doc/citing.md
yvonnefroehlich and others added 2 commits July 31, 2026 12:53
Co-authored-by: Dongdong Tian <seisman.info@gmail.com>
Comment thread README.md Outdated
Comment thread doc/team.md
@@ -3,10 +3,11 @@
We are an international team dedicated to building a Pythonic API for the Generic Mapping

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.

There are four paragraphs of long text on this page. We probably need to shortern them, but in a separate PR.

Comment thread pygmt/src/dimfilter.py Outdated
Comment thread pygmt/src/dimfilter.py Outdated
Comment thread doc/citing.md
month = jul,
year = 2026,
publisher = {Zenodo},
version = {0.19.0},

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.

Should we include the "v"? At Zenodo it is included, please see https://doi.org/10.5281/zenodo.3781524:

Tian, D., Leong, W. J., Fröhlich, Y., Grund, M., Schlitzer, W., Jones, M., Toney, L., Yao, J., Tong, J.-H., Magen, Y., Materna, K., Belem, A., Newton, T., Anant, A., Ziebarth, M., Quinn, J., He, X., Uieda, L.& Wessel, P. (2026). PyGMT: A Python interface for the Generic Mapping Tools (Version v0.19.0) [Computer software]. Zenodo. https://doi.org/10.5281/zenodo.19398871

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.

In our Zenodo records, we set the version field to "v0.19.0" rather than "0.19.0", but Zenodo suggests using a semantic version string.

image

According to the semver site, v0.19.0 is NOT a semantic version, 0.19.0 is (xref: https://semver.org/#is-v123-a-semantic-version). In other words, we should use 0.19.0 in the version field.

@yvonnefroehlich yvonnefroehlich Aug 1, 2026

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.

OK. This means we should update the metadata of the Zenodo uploads.

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.

I think so. @weiji14 any comments on it?

@yvonnefroehlich yvonnefroehlich Aug 2, 2026

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.

I believe we already had some similar discussions on this "v" during one release but could not find it at the moment in the release issues or changelog PRs.

Edit: I found it #2272 (comment) and #2272 (comment). For release 0.8.0, we decided to drop the "v" in the BibTeX entry in the README to be consistent with the CITIATION.cff file (https://github.com/GenericMappingTools/pygmt/pull/2272/changes#diff-7b3ed02bc73dc06b7db906cf97aa91dec2b2eb21f2d92bc5caa761df5bbc168f).

Currently we use the "v" in the GitHub releases and tags (https://github.com/GenericMappingTools/pygmt/tags) and the file name for the Zenodo archive as well as in the table for the Minimum Supported Versions. Probably need to add some note to the Maintainers Guide.

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.

Maybe open an issue so that we can discuss it in detail

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.

Agree, better to have a separate issue for more detailed discussions, please see #4790.

Comment thread README.md Outdated
Comment thread doc/citing.md Outdated
Comment thread pygmt/src/x2sys_init.py Outdated
Comment thread pygmt/src/x2sys_cross.py Outdated
Comment thread pygmt/src/x2sys_init.py Outdated
Comment thread pygmt/src/dimfilter.py Outdated
yvonnefroehlich and others added 4 commits August 1, 2026 19:18
Comment thread doc/citing.md Outdated
Comment thread doc/conf.py Outdated
Comment thread doc/index.md Outdated
Comment thread doc/citing.md Outdated
Comment thread doc/conf.py Outdated

@seisman seisman left a comment

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.

looks good to me

@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 Aug 2, 2026
@seisman seisman removed the final review call This PR requires final review and approval from a second reviewer label Aug 3, 2026
@seisman
seisman merged commit 9d5a904 into main Aug 3, 2026
21 of 23 checks passed
@seisman
seisman deleted the add-subsection-citing branch August 3, 2026 01:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation skip-changelog Skip adding Pull Request to changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants