-
Notifications
You must be signed in to change notification settings - Fork 28
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
Questions about Blank Nodes #89
Comments
Hi @tmciver and @koslambrou (from #106), We have two options:
The issue with the 2nd option is that the blank node label should be unique in a graph, but the
There could be a method added to the class (Generic rdfImpl, NFData rdfImpl) => Rdf rdfImpl where
...
bnodeGen :: RDF rdfImpl -> Text -> (Node, RDF rdfImpl) e.g. something like
where but this burdens all instances of Thoughts? |
@robstewart57 I don't mind option 1 with |
@tmciver and @koslambrou This is addressed in the Thoughts? |
I'd like to see a solution that guarantees blank node uniqueness for a given graph and so I'm in favor of the solution that adds a blank node creation function to the type class. But the sample code provided by @robstewart57:
suggests use of Thoughts on such a solution? |
I see that there are two constructors for blank nodes:
BNode
andBNodeGen
. When I useBNode
(orbnode
) and then serialize to Turtle, the blank nodes come through with names given by the text used to create them. Is this the intended behavior? I was expecting the more typical_:<identifier>
format.The doc string for
BNodeGen
says "An RDF blank node with an auto-generated identifier, as used in Turtle." but it does not seem get an auto-generated identifier; the identifier is simply_:genid
prepended to the integer passed into the constructor. I would expect such a feature to not require passing in an integer and to create an identifier that the library ensured was unique within the given graph. Is that the intended behavior ofBNodeGen
? Is there some other way to create a blank node with an auto-generated ID as I describe?The text was updated successfully, but these errors were encountered: