shexSpec / shex

ShEx language issues, including new features for e.g. ShEx2.1
24 stars 8 forks source link

`nodeIn` appears incorrect for blank nodes #79

Closed hsolbrig closed 5 years ago

hsolbrig commented 6 years ago

http://shex.io/shex-semantics/#values currently asserts that blank node can match any stem. As an example, if s is a LanguageStem and n is a blank node, the specification states that nodIn is satisfied if fn:startswith(identifier(n), LanguageStem.stem).

There doesn't actually seem to be enough in the grammar to allow stem matching with blank nodes.

ericprud commented 6 years ago

Agreed. [49] valueSetValue defines {iri,literal,language}Range, eliding anything like bnodeRange. There are no related tests in the test suite and I don't know of any implementations that support bnode valueSetValues.

PROPOSE: remove this text from nodeIn:

n is a blank node with identifier l and fn:starts-with(l, st).

jimkont commented 6 years ago

RESOLVED with votes from @emulatingkat, @gkellogg, @hsolbrig, @jimkont, @tombaker, @lucaswerkmeister, and @andrawaag that reacted with thumbs up on https://github.com/shexSpec/shex/issues/79#issuecomment-357547513.

Action to change the spec before we close the issue