DOC: Only link to overall Zenodo DOI.#17491
Conversation
timhoffm
left a comment
There was a problem hiding this comment.
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 <2836374+timhoffm@users.noreply.github.com>
Co-authored-by: Tim Hoffmann <2836374+timhoffm@users.noreply.github.com>
PR Summary
This prevents the page from breaking due to requesting too many DOI images from Zenodo.
Fixes #16388.
PR Checklist