DOC: replace dolphin license RDF block with prose attribution#31540
Merged
jklymak merged 1 commit intomatplotlib:mainfrom Apr 20, 2026
Merged
DOC: replace dolphin license RDF block with prose attribution#31540jklymak merged 1 commit intomatplotlib:mainfrom
jklymak merged 1 commit intomatplotlib:mainfrom
Conversation
jklymak
reviewed
Apr 20, 2026
Member
jklymak
left a comment
There was a problem hiding this comment.
Can we either change this example to not use something under license, or just cite the appropriate license rather than keep license links up to date?
Contributor
Author
Good point. I'll push an update that replaces the RDF block with a one-line prose attribution (author + "Public Domain"), so there's nothing to track long-term. Thanks for the steer! |
The inline cc:License RDF in the dolphin example was never rendered and only acted as attribution metadata, so the upstream URI namespace (web.resource.org -> creativecommons.org/ns# -> whatever comes next) is ongoing maintenance for no reader benefit. Replace the block with a single plain-text credit line naming the author and the Public Domain status. Addresses review feedback on matplotlib#31540. Refs matplotlib#30306.
e50e674 to
3954e0e
Compare
jklymak
approved these changes
Apr 20, 2026
Member
|
Thanks for the help! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR summary
Addresses one of the broken links collected in #30306 and the review
feedback from @jklymak on the original revision of this PR.
Instead of migrating the dead `web.resource.org/cc/*` URIs in
`galleries/examples/shapes_and_collections/dolphin.py` to the current
`creativecommons.org/ns#` namespace (which would have locked us into
chasing future namespace changes), the RDF block is replaced with a
single plain-text attribution line. The block was never rendered it
only carried provenance metadata in a comment so prose serves the
same purpose without the maintenance burden.
Diff
```diff
-# Dolphin from OpenClipart library by Andy Fitzsimon
-# <cc:License rdf:about="http://web.resource.org/cc/PublicDomain">
-# <cc:permits rdf:resource="http://web.resource.org/cc/Reproduction"/>
-# <cc:permits rdf:resource="http://web.resource.org/cc/Distribution"/>
-# <cc:permits rdf:resource="http://web.resource.org/cc/DerivativeWorks"/>
-# </cc:License>
+# Dolphin from OpenClipart library by Andy Fitzsimon (Public Domain).
```
PR checklist