-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
DOC: Only link to overall Zenodo DOI. #17491
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we maybe link the (or a couple of the) most recent version(s) directly and give the general link if somebody needs something else?
Plus side: It's easier for the user than to have too navigate the doi page.
Down side: We have to update it with every release (but we've done that so far as well).
If we want to put all / some of the other badges back we should probably write a helper script to automate downloading and caching them in While writing this message I talked my self into it being "easy". If I can get that working in ~30 minutes I'll open a PR with that, otherwise I'll merge this PR as a single badge is definitely better than a bunch of broken ones. |
It looks like there might also be a way to use the zenodo API to automate this further (https://developers.zenodo.org/#records), but I am over my time budget for this (my a factor of 2-3 I am terrible at estimating....) |
I added a few more links to the citation page in various places, to make it easier to find. |
Re-running this also found a few bugs in the script 😥 |
This prevents the page from breaking due to requesting too many DOI images from Zenodo. Fixes matplotlib#16388.
Co-authored-by: Tim Hoffmann <[email protected]>
Co-authored-by: Tim Hoffmann <[email protected]>
PR Summary
This prevents the page from breaking due to requesting too many DOI images from Zenodo.
Fixes #16388.
PR Checklist