-
Notifications
You must be signed in to change notification settings - Fork 555
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
dcterms no longer included among default namespaces in 6.2.0 #2103
Comments
I am still getting |
Can you provide some more context? Maybe an example?
On Mon, 20 Mar 2023 at 17:42, Ryan Shaw ***@***.***> wrote:
I am still getting Unknown namespace prefix : dcterms with rdflib-6.3.1.
—
Reply to this email directly, view it on GitHub
<#2103 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABCGC2677WQ3G4IUKCTEFBDW5CCHZANCNFSM573K6BAA>
.
You are receiving this because you modified the open/close state.Message
ID: ***@***.***>
--
Iwan Aucamp
|
The example I provided above for 6.2.0 still fails with 6.3.1. |
@rybesh I see now, the problem is, the default set of bindings after #1686 is "core", but this excludes most of the bindings that were present in 6.1.1. I think we probably should switch the default from "core" to "rdflib", @nicholascar it would be good to have your input here. |
@cmungall would be good to get your input on this also. |
I started this discussion: https://groups.google.com/g/rdflib-dev/c/2Dg7MkldV1c Unless there is some community consensus I'm going to close this as wontfix, I'm really sorry we missed it, but the damage is done and I fear changing back to what it was will just upset everyone from scratch. I will also keep working to improve our process and quality assurance to eliminate problems like this in future. |
|
I will see where the discussion goes, if I can get feedback from @nicholascar and @cmungall and/or consensus on that mailing list discussion I will fix the regressions. |
I preferred to have more common namespaces bound by default, so "rdflib" instead of "core", but there was push-back on pre-binding many from some (I think more than one!) commenters on the update. However, I still think that binding more is ok, as long as 1. what is bound is known and 2. things can be overridden. Both of these things are true: the NamespaceManager records all prefixes registered for a graph and things can be overridden with So I still support binding according to "rdflib" by default, not "core". |
Given the feedback [ref] I will do the following for 6.3.2
|
The namespaces bound by default by `rdflib.graph.Graph` and `rdflib.namespace.NamespaceManager` was reduced in version 6.2.0 of RDFLib, however, this also would cause code that worked with 6.1.1 to break, so this constituted a breaking change. This change restores the previous behavior, binding the same namespaces as was bound in 6.1.1. To bind a reduced set of namespaces the `bind_namespaces` parameter of `rdflib.graph.Graph` or `rdflib.namespace.NamespaceManager` can be used. - Closes <RDFLib#2103>.
The namespaces bound by default by `rdflib.graph.Graph` and `rdflib.namespace.NamespaceManager` was reduced in version 6.2.0 of RDFLib, however, this also would cause code that worked with 6.1.1 to break, so this constituted a breaking change. This change restores the previous behavior, binding the same namespaces as was bound in 6.1.1. To bind a reduced set of namespaces the `bind_namespaces` parameter of `rdflib.graph.Graph` or `rdflib.namespace.NamespaceManager` can be used. - Closes <RDFLib#2103>.
The namespaces bound by default by `rdflib.graph.Graph` and `rdflib.namespace.NamespaceManager` was reduced in version 6.2.0 of RDFLib, however, this also would cause code that worked with 6.1.1 to break, so this constituted a breaking change. This change restores the previous behavior, binding the same namespaces as was bound in 6.1.1. To bind a reduced set of namespaces the `bind_namespaces` parameter of `rdflib.graph.Graph` or `rdflib.namespace.NamespaceManager` can be used. - Closes <RDFLib#2103>.
The namespaces bound by default by `rdflib.graph.Graph` and `rdflib.namespace.NamespaceManager` was reduced in version 6.2.0 of RDFLib, however, this also would cause code that worked with 6.1.1 to break, so this constituted a breaking change. This change restores the previous behaviour, binding the same namespaces as was bound in 6.1.1. To bind a reduced set of namespaces, the `bind_namespaces` parameter of `rdflib.graph.Graph` or `rdflib.namespace.NamespaceManager` can be used. - Closes <#2103>.
rdflib 6.1.1:
rdflib 6.2.0:
The text was updated successfully, but these errors were encountered: