-
Notifications
You must be signed in to change notification settings - Fork 565
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
Graphs don't serialize new content correctly #228
Comments
Wrong usage of the API, subjects can not be plain text variables. Therefore you would need to use something like:
Maybe we would need to improve both the documentation and the feedback provided to the user on such situations... |
Okay, it makes sense to combine this with #227 also and add some more sensible valuechecking in the add method. Not only should it reject plain strings, it should also reject triplets which have the subject and predicate as unicode objects. |
This is the same underlying issue as #200 again. @wikier - actually no part of the triple (or quad, see #167) should be a string, it may work sometimes, but will come back and bit you sooner or later. "unicode object has no .n3()" method for instance... We could "auto-box" non rdflib.term.Node objects though - but only in rdflib4, where a literal/python mapping overhaul is planned anyway. |
I made #230 for the possibility of auto-boxing objects. |
Consider the following:
My new triplet is not there.
Ah you say, but you should
commit
your graph! But that also does not work:What is going on here? Note that in both cases the Namespace does get added to the graph, so clearly something is happening.
The text was updated successfully, but these errors were encountered: