Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add PyGMT Zenodo BibTeX entry to main README.md #678

Merged
merged 4 commits into from
Nov 3, 2020
Merged

Conversation

weiji14
Copy link
Member

@weiji14 weiji14 commented Nov 3, 2020

Description of proposed changes

To ensure that people will cite PyGMT consistently in their research work, this PR adds a PyGMT BibTeX entry to the main README.md (copied from Zenodo at https://zenodo.org/record/4025418/export/hx).

Also point to https://www.generic-mapping-tools.org/cite for GMT specific modules.

Fixes #653

Reminders

  • Run make format and make check to make sure the code follows the style guide.
  • Add tests for new features or tests that would have caught the bug that you're fixing.
  • Add new public functions/methods/classes to doc/api/index.rst.
  • Write detailed docstrings for all functions/methods.
  • If adding new functionality, add an example to docstrings or tutorials.

Notes

  • You can write /format in the first line of a comment to lint the code automatically

@weiji14 weiji14 added the documentation Improvements or additions to documentation label Nov 3, 2020
@weiji14 weiji14 added this to the 0.2.1 milestone Nov 3, 2020
@seisman
Copy link
Member

seisman commented Nov 3, 2020

What about a stand-alone CITATION.md file in the repository, see GenericMappingTools/gmt#115.

Question: Do we point to https://www.generic-mapping-tools.org/cite as well?

Yes, I think we should encourage users to cite both PyGMT and GMT6 if possible.

@weiji14
Copy link
Member Author

weiji14 commented Nov 3, 2020

What about a stand-alone CITATION.md file in the repository, see GenericMappingTools/gmt#115.

Don't think we need a dedicated page for this now since it's just one entry. We can point to https://pygmt-98df8j9um.vercel.app/#citing-pygmt (or https://pygmt.org/#citing-pygmt when this PR is merged) to instruct on how to cite PyGMT.

Yes, I think we should encourage users to cite both PyGMT and GMT6 if possible.

👌

@seisman
Copy link
Member

seisman commented Nov 3, 2020

The line breaks in the title and note are unnecessary:
image

@seisman
Copy link
Member

seisman commented Nov 3, 2020

Don't think we need a dedicated page for this now since it's just one entry. We can point to https://pygmt-98df8j9um.vercel.app/#citing-pygmt (or https://pygmt.org/#citing-pygmt when this PR is merged) to instruct on how to cite PyGMT.

I'm feeling that the README.md (or the website landing page) is a little too long, and readers may miss the citation information.

@weiji14
Copy link
Member Author

weiji14 commented Nov 3, 2020

Don't think we need a dedicated page for this now since it's just one entry. We can point to https://pygmt-98df8j9um.vercel.app/#citing-pygmt (or https://pygmt.org/#citing-pygmt when this PR is merged) to instruct on how to cite PyGMT.

I'm feeling that the README.md (or the website landing page) is a little too long, and readers may miss the citation information.

I suppose it depends on what's better for the search engine (e.g. results from "How to cite PyGMT"). There's some tips at http://citeas.org/modify-your-citation and http://citeas.org/sources. Currently looks like this at http://citeas.org/cite/pygmt:

image

It's a lot better for GMT at https://citeas.org/cite/gmt:

image

Personally I just do CTRL+F and type 'cite' or 'BibTeX' to find the citation info, but we'll need to optimize this for the machines 🤖 too 😄

README.rst Outdated Show resolved Hide resolved
@weiji14
Copy link
Member Author

weiji14 commented Nov 3, 2020

Another argument for having the BibTeX in the main README.md is that it will appear on PyPI as well 😉

@weiji14 weiji14 marked this pull request as ready for review November 3, 2020 22:47
@weiji14 weiji14 merged commit e14c614 into master Nov 3, 2020
@weiji14 weiji14 deleted the bibtex/pygmt-v0.2.0 branch November 3, 2020 22:50
@seisman
Copy link
Member

seisman commented Nov 3, 2020

Forgot to add "updating citation information" in the release checklist.

@weiji14
Copy link
Member Author

weiji14 commented Nov 3, 2020

Oh right, I'll merge in #673 first and we can do that, and probably start the changelog for PyGMT v0.2.1

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

How to cite PyGMT
2 participants